[Plugin: WP-Invoice – Web Invoice and Billing] Not quite ready for for my needs.. Too bad.. looks li

DivaVocals,

why we should update our plugin due to the issue caused by another plugin?

We are trying to fix everything we can fix. So, why we couldn’t fix this after almost a year from reporting? We would do that if we could. But be sure please, we are trying to find a way.

The main cause of the issue is that SFC developers use standard wordpress filters in their plugin. This is bad.

remove_filter( 'the_content', 'wptexturize' );
$text = apply_filters('the_content', $text);
add_filter( 'the_content', 'wptexturize' );

They do apply_filters of ‘the_content’, which should be called by wordpress only.

**sigh**

Anton.. re-read my post.. I honestly don’t care who fixes what as long as the issue gets addressed. Now if that means means you guys make a change to your plugin or the other plugin makes a change or BOTH plugins make a change.. it truly doesn’t matter.. Speaking for myself and anyone else with the issue, we just wanted a solution.. we now have a solution and for now this will resolve the issue..

Sooooooooo to answer your question, I never said specifically that YOU had to fix anything..

Hopefully the SFC devs will fix their plugin, but what you guys CAN do (and I’ve seen this with other plugins) is add this as part of your user docs so that user’s of your plugin are made aware of the conflict with SFC and the fix they need to do to overcome it..

@phil@silverstar-media.com

I tried your fix and no joy.. In order for partial payments to work I still have to turn off SFC.. Should the word “invoicing” be invoice or something else??

DivaVocals,

I think we have good new for you. We did a deep investigation of the problem. The fix we did should be tested to be sure it is stable for different installations.

Could you please give the email where I can send the file with the fix and information on how to set it up?

Thank you.

@divavocals

It’s checking for the word “invoicing” in the URL. If you’ve changed the default page title or permalink for the invoicing page, you’ll just simply need to use the new title/permalink. For instance, if you renamed the page to “invoice”, you’ll simply use that instead of “invoicing”.

It does seem that the devs may have come up with a workable patch, so test that out and let us know how it works out.

Anton..

That sounds very promising.. ??

I don’t want to post my e-mail address on the public forum.. If you are okay with posting your e-mail address, please provide it and I will send you an e-mail from the address where you can send me what you have..

@ [email protected]
My URL looks like this:
https://mydomain.com/invoices/?invoice_id=#######################

So would I use “invoices”

DivaVocals,

[email protected]

Thanks.

You’ve got mail.. ??

Tested and this did not appear to work.. Partial payments only work when I turn off SFC..

Can I have the link of the invoice to see?

@divavocals

the fix helped another person. Please be sure you replace correct file in correct place

@anton.. Since I used the file comparison program “Beyond Compare” to compare your file to the one on my LIVE site, I am indeed positive that I placed the code in the right file and in the right place:

wp-content/plugins/wp-invoice/core/wpi_ui.php

I will again state that the change did not work.. Here’s a test invoice where you can see that it doesn’t work:

https://overthehillweb(dot)com/invoices/?invoice_id=a358b9a31ce245dca665005524fea8f8

Diva,

When you tried the fix I suggested, did you change this ‘invoicing’ to ‘invoices’?:

if( substr($_SERVER["REQUEST_URI"], 0, 11) != '/invoicing/' ) {
		include 'sfc-base.php';
	}

to this:

if( substr($_SERVER["REQUEST_URI"], 0, 11) != '/invoices/' ) {
		include 'sfc-base.php';
	}

Also, it looks like you’re using the Simple Facebook Connect widget on the invoice page… Not sure if it’s needed on that page, but that could also be causing a conflict.

@divavocals,

this is another sort of conflict which we have already fixed about month ago. It is included into RC. Will be pushed this week, hopefully today.