Closed (fixed)
Project:
Feeds XPath Parser
Version:
7.x-1.0-beta3
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
12 Jan 2012 at 21:12 UTC
Updated:
12 Jan 2013 at 02:14 UTC
Jump to comment: Most recent file
Comments
Comment #1
Robin Millette commentedI'm sort of new to Drupal 7 (been using Drupal since 2005, and Feed API, then Feeds, etc.) but with ctools 7.x-1.0-rc1 plugins must end with ".class.php" to be detected.
I tried that, renaming the 5 FeedsXPathParserXXX.inc files to FeedsXPathParserXXX.class.php and updating feeds_xpathparser.info to reflect the change. No more errors, but note that I haven't even tried using Feeds, let alone xpathparser yet. I'll keep this issue updated if I find anything else.
In fact, searching recent commits on ctools, this is an old change from 2009:
http://drupalcode.org/project/ctools.git/commitdiff/800708cfc93d9ea82461...
I could be missing something obvious...
Comment #2
misc commentedSame problem here. A patch is attached.
Comment #3
misc commentedMissed to add the changes to the .info, corrected and updated the patch.
Comment #4
twistor commentedAlright, I fixed this here, http://drupalcode.org/project/feeds_xpathparser.git/commit/4b1f1a4.
As a note: This change was originally added do to #1265564: PDOException due to class auto-loading registry when running a cache clear.
Comment #6
davmorr commentedAlthough this issue is closed and a viable and sensible patch submitted, I thought that it would be worth pointing out that this error can be mitigated by using the handler -> file key in the plugins array returned by feeds_xpathparser_feeds_plugins() in the module file: feeds_xpathparser.module
The only real benefit here, at least that I am aware of, is that it keeps in line with the plugins file naming convention established by the Feeds module, see feeds/feeds.plugins.inc .
Comment #7
twistor commentedThat was the fix that was committed, I did not rename the files.
Comment #8
davmorr commentedAh, okay. I didn't see that in the dev version posted as of 12/3/12, I was just going by what was posted in this thread, as I thought that was the accepted solution . Thanks!
BTW, the fix that I posted in this thread worked fine on my local machine, but when we applied it to our production server the path to the class was duplicating the dir path; i.e. /sites/all/modules/feeds_xpathparser/sites/all/modules/feeds_xpathparser/FeedsXPathParserHTML.inc
When I removed the prefixed $path var (below), the path resolved properly.
$path = drupal_get_path('module', 'feeds_xpathparser');
...,
'file' => $path . '/FeedsXPathParserHTML.inc',
Cheers!
Comment #9
jmomandown commentedWhere are you seeing this dev version? It is not listed via the project page or the repository