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.
hi
Path in configuration option for js is wrong.
its include sites/all/. So in drupal 8 we have libraries directory is in root directory.
thanks
Comment | File | Size | Author |
---|---|---|---|
#2 | superfish-path_for_js_is_incorrect-2647240-1-drupal8rc2.patch | 7.64 KB | Sumit kumar |
Comments
Comment #2
Sumit kumar CreditAttribution: Sumit kumar as a volunteer commentedadding patch for it
Comment #3
Sumit kumar CreditAttribution: Sumit kumar as a volunteer commentedComment #4
nicholas.alipaz CreditAttribution: nicholas.alipaz at L.A. BioMed commentedThe path is not technically incorrect for being in /sites/all. However for me, I have superfish under /modules/contrib/superfish which means the references to the relative path ../../sites/all does not work properly and the files don't load. I changed to /sites/all in my libraries.yml file and everything is fixed. I don't really mind whether this module decides to look for the libraries in /sites/all/libraries or in /libraries.
Comment #5
therealssj CreditAttribution: therealssj commentedIn the documentation it has the default directory as 'sites/all/libraries' so if we change this here we ought to change it there also.
Though it really doesn't affect anything as in the end this path can be configured through the configuration page.
Comment #6
nicholas.alipaz CreditAttribution: nicholas.alipaz at L.A. BioMed commentedMy comment is in regards to the relative directory reference being incorrect...
The code looks for the library at ../../sites/all/libraries However, that will not work if the module is installed in a subdirectory of modules folder. this needs to be fixed.
Comment #7
therealssj CreditAttribution: therealssj commented@nicholas.alipaz the configuration page lets you define the directory of the library, did you try using that?
I havent tried messing with it so not sure if it works correctly.
Comment #8
nicholas.alipaz CreditAttribution: nicholas.alipaz at L.A. BioMed commentedtherealssj, hmm I hadn't noticed that. However, if I remember correctly, I couldn't actually install the module since it won't install when the library is missing (or maybe I am remembering this wrong). I had to edit http://cgit.drupalcode.org/superfish/tree/superfish.libraries.yml?id=c90... to remove all the relative paths in order to install and still keep my module at `/modules/contrib/superfish`. I changed all paths to be from the root, `/sites/all/libraries/...`
Comment #9
geek.geek CreditAttribution: geek.geek as a volunteer commented(the "configuration" page for this module has been removed; automatic library path detection was added instead)