On line 266 include function is used in the commerce_extra_price_formatters.module file.

Please use module_load_include instead.

I created a patch file for this.

CommentFileSizeAuthor
commerce_extra_price_formatters.module.patch663 bytesskipyT

Comments

pebosi’s picture

pebosi commented
Status: Active » Needs review

Patch works for me. Thanks

farald’s picture

farald commented
Status: Needs review » Reviewed & tested by the community

Me too.

ikos’s picture

ikos commented

Hi,

Thanks for submitting this.

I have been working on a few changes to the module which I need to bundle up into a release. However I'll commit this patch tonight just to clear the problem some of you are experiencing.

kind regards

Richard

skipyT’s picture

skipyT commented

Hi,

It would be nice!

Thanks!

r2integrated’s picture

r2integrated commented

Confirming patch works. Thanks @skipyT =)

ikos’s picture

ikos commented
Status: Reviewed & tested by the community » Fixed

Patch committed. Thanks!

kind regards

Richard

Ivan Simonov’s picture

Ivan Simonov commented
Status: Fixed » Needs review

module_load_include make an extra DB query to find path for module and then call require_once function.
Actually we do not need this path.

We can use just:
require_once('includes/extra_functions.inc');

It works on my installation.

Ivan Simonov’s picture

Ivan Simonov commented
Status: Needs review » Fixed

OK

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.