Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
PEAR Cache_Lite have Cache.php, I use Cache_Lite in modules,
and because of line
require 'Cache.php';
in CacheRouter.php it is problem - cacherouter include Cache.php from PEAR package, if I change include_path Cache_Lite include Cache.php from cacherouter and it is problem again.
require drupal_get_path('module', 'cacherouter').'/Cache.php';
didn't work, because of cacherouter design.
and option
require getcwd().'sites/all/modules/cacheroter/Cache.php';
can not be provided as patch because of hardcoded path.
Any ideas how I can do acceptable patch for this issue?
Comment | File | Size | Author |
---|---|---|---|
#3 | CacheRouter.patch | 424 bytes | Alexander Kosarev |
#2 | CacheRouter.patch | 119 bytes | Alexander Kosarev |
Comments
Comment #1
andypostneed more info because CR uses
maybe your bug related #564460: Class conflict, namespace collisions
Comment #2
Alexander Kosarev CreditAttribution: Alexander Kosarev commentedsuggestion solution work just fine
I have did it as patch
Comment #3
Alexander Kosarev CreditAttribution: Alexander Kosarev commentedprevious path was created by "diff file file"
this one is by "diff -up file file"
Comment #4
andypost@Alexander You does not understand me, cacherouter includes files from it's own folder.
dirname(__FILE__) returns cachetouter's folder, it's the same as
require './Cache.php';
Please try 5-dev version which already fixed, your patch against cvs will break things
require dirname(__FILE__) .'/Cache.php';
Your problem caused by 5-beta9 which is
require 'Cache.php';
Comment #5
slantview CreditAttribution: slantview commentedThis is a duplicate of #564460: Class conflict, namespace collisions
Comment #6
Alexander Kosarev CreditAttribution: Alexander Kosarev commentedwith DEV version this problem solved.
thanks.
Comment #7
andypostThis is dup