On trying to install FeedAPI on Drupal 5.1. Have also replicated the issue on a clean install of Drupal 5.7. MySQL DB version is 4.0.27 and PHP version is 4.4.7. Pasting the error below and including a screenshot. The result is that the necessary tables aren't created, so the module doesn't function.

user warning: You have an error in your SQL syntax. Check the manual that corresponds to your MySQL server version for the right syntax to use near 'character set latin1 collate latin1_general_ci NOT NULL, query: CREATE TABLE feedapi_node_item ( fiid int(10) unsigned NOT NULL, feed_nid int(10) unsigned NOT NULL, nid int(10) unsigned NOT NULL, url text character set latin1 collate latin1_general_ci NOT NULL, `timestamp` int(10) unsigned NOT NULL default '0', arrived int(10) unsigned NOT NULL default '0', guid text character set latin1 collate latin1_general_ci NOT NULL, PRIMARY KEY (nid), KEY fiid (fiid), KEY feed_nid (feed_nid), KEY arrived (arrived), KEY url (url(255)), KEY guid (guid(255)) ) DEFAULT CHARSET=latin1; in /homepages/12/d203637805/htdocs/sances/includes/database.mysql.inc on line 172.

CommentFileSizeAuthor
#1 feedapi_installerror.png37.86 KBMike Sances

Comments

Mike Sances’s picture

Mike Sances commented
StatusFileSize
new feedapi_installerror.png37.86 KB

screenshot didn't attach, so trying again...

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented
Status: Active » Closed (won't fix)

Let's see the Drupal system requirements:
http://drupal.org/requirements

MySQL 4.1 or higher are strongly recommended.

I tested FeedAPI before the release on MySQL 5 series and on MySQL 4.1.11 .
Sorry, I do not plan to modify the code, I suggest you a workaround:
Remove all the character code-specific stuff.
So you should see this in feedapi_node.install

CREATE TABLE {feedapi_node_item} (
        fiid int(10) unsigned NOT NULL,
        feed_nid int(10) unsigned NOT NULL,
        nid int(10) unsigned NOT NULL,
        url text NOT NULL,
        `timestamp` int(10) unsigned NOT NULL default '0',
        arrived int(10) unsigned NOT NULL default '0',
        guid text NOT NULL,
        PRIMARY KEY  (nid),
        KEY fiid (fiid),
        KEY feed_nid (feed_nid),
        KEY arrived (arrived),
        KEY url (url(255)),
        KEY guid (guid(255))
      )

It should work in this way. (sorry, i could not try out, i have no access to mysql < 4.1)

Mike Sances’s picture

Mike Sances commented

Thanks for your quick and helpful reply. I did try what you suggested, it seemed to install fine and tables were created from what I could see in phpmyadmin. However, when I go to submit a feedapi content type (both lightweight and normal), I get a blank page on submitting, and no rows are created in the mysql database. No errors are recorded in watchdog.

I did note the Drupal "requirements" page, but "strongly recommend" is not the same as requirements. Would you consider specifying in the FeedAPI docs that only MySQL 4.1 above are supported?

Mike Sances’s picture

Mike Sances commented
Priority: Critical » Minor

I upgraded to php 5 and mysql 5 and now am rolling fine with your module. Thanks.