Hi folks,
I have just tried activating Amazon on Drupal 6.4
I get this error:

Fatal error: Call to undefined function content_notify() in /public_html/cms/modules/amazon/asin/asin.install on line 11

I see this error was logged for the previous beta of Amazon and then closed due to inactivity (I added a comment to that bug tread also). So I guess it's not that common, but is still occurring.

Any ideas?
Cheers,
Jonathan

CommentFileSizeAuthor
#3 asin.info_.diff346 bytesAnonymous (not verified)

Comments

inspiredearth’s picture

The previous error message was generated by the server (not displayed within Drupal).
I've now returned to listing modules and go these errors reported by Drupal:

* warning: include_once(.//content.module) [function.include-once]: failed to open stream: No such file or directory in /home/publish/public_html/cms/modules/amazon/asin/asin.install on line 5.
* warning: include_once() [function.include]: Failed opening './/content.module' for inclusion (include_path='.:/usr/lib/php:/usr/local/lib/php') in /home/publish/public_html/cms/modules/amazon/asin/asin.install on line 5.
* warning: include_once(.//content.module) [function.include-once]: failed to open stream: No such file or directory in /home/publish/public_html/cms/modules/amazon/asin/asin.install on line 5.
* warning: include_once() [function.include]: Failed opening './/content.module' for inclusion (include_path='.:/usr/lib/php:/usr/local/lib/php') in /home/publish/public_html/cms/modules/amazon/asin/asin.install on line 5.

When I refresh the modules list, the error messages have gone, but only the Amazon API is activated and the other part parts are not.

Cheers,
Jonathan

Anonymous’s picture

I got this same error when trying to install the API and all the other Amazon modules at the same time.

Anonymous’s picture

StatusFileSize
new346 bytes

I just figured out what's going on here -- Amazon Field requires CCK, which I did not have installed. content_notify() is provided by CCK.

However, Amazon Field does not list any dependencies. The asin/asin.info file erroneously uses requirements[] instead of dependencies[]. I've attached a patch to correct this.

Anonymous’s picture

Bump.

Seriously, this is still open after six months? It's a 10-second fix. See the diff in my previous comment.

greggles’s picture

Status: Active » Needs review

Proper status can help.

eaton’s picture

Status: Needs review » Fixed

Committed. Thanks for the catch.

Status: Fixed » Closed (fixed)

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

Aabuhamedah’s picture

Assigned: Unassigned » Aabuhamedah