RSS Aggregator bug

[colas]

I have set up a new S9Y install.
I installed via spartacus the RSS Aggregator plugin.
Each time I try to add a RSS feed I get the SQL errors printed (twice) on top of the page:

Warning: sqlite_query() [function.sqlite-query]: ambiguous column name: categoryid in /home/www/htdocs/s9y/include/db/sqlite.inc.php on line 226

This happens in 1.0.3 and 1.0.4a, on 2 different installs.

Does this ring a bell? Is there some known incompatible plugin? Does it work with sqlite?


I turned debug=true in sqlite.inc.php, and I have in the sqlite.log file for the 2 erroneous calls:


[11.12.2006 16:51] SQLITE QUERY: SELECT serendipity_aggregator_feeds.feedid,
feedname, feedurl, htmlurl, categoryid, last_update, charset
FROM serendipity_aggregator_feeds
LEFT OUTER JOIN serendipity_aggregator_feedcat
ON serendipity_aggregator_feeds.feedid = serendipity_aggregator_feedcat.feedid
ORDER BY feedname, feedid
[11.12.2006 16:51] [ERROR]

[garvinhicking]

Hi!

Thanks a lot. It seems SQLite is not as forgiving as MySQL in that case. I just committed a new version of the plugin:

http://php-blog.cvs.sourceforge.net/php ... 58&r2=1.59

HTH,
Garvin

[colas]

Thanks! However...

I deleted the plugin via the web interface, re-installed it (an upgrade was detected),
upgrade went ok, but on clicking the "RSS Aggregator" menu item in the left box,
I got the following warnings, and then I have the same problems as above:


Warning: sqlite_query() [function.sqlite-query]: table serendipity_aggregator_feeds already exists in /home/www/htdocs/s9y/include/db/sqlite.inc.php on line 226
*** setup DB version

Warning: sqlite_query() [function.sqlite-query]: near "md5": syntax error in /home/www/htdocs/s9y/include/db/sqlite.inc.php on line 226

Warning: sqlite_query() [function.sqlite-query]: no such table: serendipity_aggregator_md5 in /home/www/htdocs/s9y/include/db/sqlite.inc.php on line 226
*** setup DB version 2

Warning: sqlite_query() [function.sqlite-query]: table serendipity_aggregator_feedcat already exists in /home/www/htdocs/s9y/include/db/sqlite.inc.php on line 226

Warning: sqlite_query() [function.sqlite-query]: index feedid_idx already exists in /home/www/htdocs/s9y/include/db/sqlite.inc.php on line 226

Warning: sqlite_query() [function.sqlite-query]: serendipity_aggregator_feedcat.categoryid may not be NULL in /home/www/htdocs/s9y/include/db/sqlite.inc.php on line 226

Warning: sqlite_query() [function.sqlite-query]: near "ALTER": syntax error in /home/www/htdocs/s9y/include/db/sqlite.inc.php on line 226

Warning: sqlite_query() [function.sqlite-query]: near "ALTER": syntax error in /home/www/htdocs/s9y/include/db/sqlite.inc.php on line 226

Warning: sqlite_query() [function.sqlite-query]: ambiguous column name: categoryid in /home/www/htdocs/s9y/include/db/sqlite.inc.php on line 226
RSS Aggregator

[garvinhicking]

Hi!

You don't need to reinstall a plugin to upgrade; just upload the updated PHP file to your directory.

It seems that the aggregator plugin uses the 'ALTER TABLE' statement to perform some DB updates, but SQLite doesn'T support alter table. Thus I'm afraid that this plugin is not really compatible with SQLite. Most of our plugins are, but some that use ALTER TABLE (maybe 5% of our plugins) may not work fully on SQLite.

However I don'T understand why the ambiguous column name error still appears - this should have been fixed by my patch. Are you sure you downloaded the version of the plugin properly? You must now be running version 0.13 of the plugin?

REgards,
garvin

[colas]

I re-downloade the php by hand, it says that it is the 0.13 version, but
I still have the same ambiguous error messages.

Anyways, I thing the proper fix is just to document in a very visible way
in the description that this plugin is incompatible with SQLite, it will
then just use a standalone RSS Aggregator, (like gregarious) no problem.

[garvinhicking]

Hi!

I want to get at least the ambiguous thing solved *g* -- so, did you re-enable the debugging to see how the query now looks like that causes the ambiguous-error? It should use the "f.categoryid" statement there...?

Regards,
Garvin