Combine processor not replacing

  • Resolved Avf

    (@avf)


    8 years, 11 months ago

    Hello and thanks for this great plugin.
    My question is why when I use the combine processor, the result is the same name of the variable, not the contents?
    When inserting {po[1] full_html} as per instructions, it shows exactly the same {po[1] full_html} as output, and not the contents of po[1] full_html.

Viewing 9 replies - 1 through 9 (of 9 total)
  • Plugin Contributor Tung Pham

    (@phamtungpth)

    8 years, 11 months ago

    Hi,

    Which version of WPPipes are you using? Is it the latest?

    Please export your pipe and upload it onto a share host (such as Dropbox) then provide me the link! I will try to check how it works on my demo site!

    Best Regards!

    Thread Starter Avf

    (@avf)

    8 years, 11 months ago

    WP version 4.7.5
    WP-Pipes version 1.25.
    I created a test pipe, you can download it from https://1drv.ms/u/s!Aiveo2dI6XKtoWZ9LKi60ugRj4Aw . I’ve never got the combine processor to work, for some reason (maybe a php mod missing from my server?).
    You can check the feed created by the pipe at https://www.fontoura.com/feeds/testing.rss and notice the description is not correct, as below:

    		<item>
			<title>Auditores fiscais entram em greve e fazem operação padrão nas aduanas</title>
			<link>https://www.fontoura.com/noticias/fontes/agencia-brasil/auditores-fiscais-entram-em-greve-e-fazem-operacao-padrao-nas-aduanas/</link>
			<description>{po[0] html}: {pi[1] html}
{po[1] introtext}</description>
			<dc:creator>25458156c5222d74956599335c97200e</dc:creator>
			<guid isPermaLink="false">https://www.fontoura.com/noticias/fontes/agencia-brasil/auditores-fiscais-entram-em-greve-e-fazem-operacao-padrao-nas-aduanas/</guid>
		</item>

    Thanks again.
    Regards

    • This reply was modified 8 years, 11 months ago by Avf.
    • This reply was modified 8 years, 11 months ago by Avf.
    • This reply was modified 8 years, 11 months ago by Avf.
    Plugin Contributor Tung Pham

    (@phamtungpth)

    8 years, 11 months ago

    Hi,

    I run your pipe on my demo pipe and the Combine processor works. Please watch this https://www.screencast.com/t/mCfwGMQaiY

    I only removed {pi[1] html} in Combine processor because that tag is not usable. You could only use tags with the output fields of Source add-on and other processors. {pi[1] html} is the input field, not output.

    Please check WPPipes Requirements in the settings page of WPPipes on your site!

    Thread Starter Avf

    (@avf)

    8 years, 11 months ago

    Thanks for your reply.
    That’s very interesting, even without pi it doesn’t work. All requirements are satisfied, however we are running php 7.
    Can you please provide more information on the technical side of this processor, so we can know where to debug? Maybe the php file where the processor is.
    I’m thinking it could be an opcache issue.

    Plugin Contributor Tung Pham

    (@phamtungpth)

    8 years, 11 months ago

    Hi,

    The Combine processor’s code is located at YOUR_PATH_SITE/wp-content/plugins/wp-pipes/plugins/processors/combine/combine.php file.

    Best Regards!

    Thread Starter Avf

    (@avf)

    8 years, 11 months ago

    Thanks!
    Ok, debugging that file, I get the following error for each tag that should be replaced:

    “Notice: Array to string conversion in /wp-content/plugins/wp-pipes/plugins/processors/combine/combine.php on line 37”

    Line 37 is
    if(isset($data->no_need['op'][$result[0]]->$seperate_array[1])){

    Any idea on what’s going on?

    Thread Starter Avf

    (@avf)

    8 years, 11 months ago

    It seems this code 

    preg_match('/(?<=\[).*?(?=\])/i', $seperate_array[0], $result);
if(isset($data->no_need['op'][$result[0]]->$seperate_array[1])){
	$inputs[$matches[0][$key]] = $data->no_need['op'][$result[0]]->$seperate_array[1];
}

    is not compatible with php7.

    • This reply was modified 8 years, 11 months ago by Avf.
    Plugin Contributor Tung Pham

    (@phamtungpth)

    8 years, 11 months ago

    Hi,

    Thank you for informing!

    I just release new version of WPPipes. With that version, I did some changes in Combine processor. Hopefully it works on your PHP7.

    Best Regards!

    Thread Starter Avf

    (@avf)

    8 years, 11 months ago

    It works now with the fix. Thanks!

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

The topic ‘Combine processor not replacing’ is closed to new replies.