WPIntell

Source evidence

Fatal Error – Class Name Collisions

Sofortueberweisung Gateway for Woocommerce · support · 2020-01-08T04:39:00+00:00

mixedsentiment
highseverity
0.88relevance
3replies
Evidence linked to opportunitycommercial context

Proof Health

Open evidence

Commercial opportunities need traceable source links before they are treated as build-worthy.

10 / 39 rows with source links

25.6% of this page's analysis has direct source links.

0 build-decision rows missing links

0 rows here require auditable proof before promotion.

29 rows with no attached evidence

0 rows have source counts but still need direct links.

Conversation

support
ollybach resolved
Hi maybe i am wrong of course, but it looks like the plugin includes/requires the sofort libraries and classes all the time / everywhere instead of only where they are needed. If someone has installed 2 different shop systems (i.e woocommerce and another one) but both have a sofort checkout process and use/bundle the sofort libraries, you end up with fatal errors like class already declared or similar. Unless I am mistaken, and please correct me if I’m wrong – may i respectfully suggest to only require/include the sofort libs where and when they are needed please? (i.e just before the new Sofortueberweisung($configkey) and wherever else is appropriate instead of globally) thanks This topic was modified 6 years, 4 months ago by ollybach . Hello @ollybach i will check that. Cheers, Michael Hello, in the next version the sofort lib are only loaded if needed (process_payment). The new version should be released end of this week. Cheers, Michael Hello, sorry this makes for me no sense because new Sofortueberweisung($configkey) is called only on process_payment() (WooCommerce Hook). So if you use 2 different Shop/Ecommerce Plugins – there should no problems – right? Cheers, Michael

Comments

3 shown
mlfactory 2020-01-09T16:31:00+00:00

Hello @ollybach i will check that. Cheers, Michael

mlfactory 2020-03-25T10:32:00+00:00

Hello, in the next version the sofort lib are only loaded if needed (process_payment). The new version should be released end of this week. Cheers, Michael

mlfactory 2020-05-24T15:07:00+00:00

Hello, sorry this makes for me no sense because new Sofortueberweisung($configkey) is called only on process_payment() (WooCommerce Hook). So if you use 2 different Shop/Ecommerce Plugins – there should no problems – right? Cheers, Michael