Viewing 5 replies - 1 through 5 (of 5 total)
  • Plugin Author Allen

    (@amweiss98)

    what file are you speaking of…can you be a bit more specific – like in the feed to post or shortcode?

    Thread Starter BuckShank

    (@buckshank)

    [wp-rss-multi-importer]$ grep -r “\$attribution” *
    inc/import_posts.php: $attribution=’: ‘;
    inc/import_posts.php: $attribution=$options[‘sourcename’].’: ‘;
    inc/rss_feed.php: $attribution=’: ‘;
    inc/rss_feed.php: $attribution=$options[‘sourcename’].’: ‘;

    Not a programmer are you?

    Plugin Author Allen

    (@amweiss98)

    I’m the ceo of an internet company who happens to do some programming (I have a staff of many programmers so I don’t call myself a programmer). I’ll take a look at the code – the shortcode should work as an override if you have the attribution text box blank.

    Thread Starter BuckShank

    (@buckshank)

    Here is an example of the flawed logic even though programmatically correct. The idea did not make it into the code.

    From the feed_to_post settings page

    Feed Source Attribution Label –> Other fill in below

    Your own attribution label –>My label (make sure to choose Other in drop down list)

    From inc/import_posts.php

    This code will tack a “:” on my atribution label.

    if(empty($options[‘sourcename’])){
    $attribution=”;
    }else{
    $attribution=$options[‘sourcename’].’: ‘;
    }

    The colon should be part of the default text strings from the default list; not programmatically attached later. If anything should be attached; it should be a space ‘ ‘;.

    Plugin Author Allen

    (@amweiss98)

    ok..take out the colon in the code and I’ll have a space in the next update.

Viewing 5 replies - 1 through 5 (of 5 total)

The topic ‘$attribution default set incorrectly’ is closed to new replies.