Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
I have the latest Libraries release version and the latest Commons dev. This has in occurred in the last couple of dev releases on both windows and nix. I'm not sure if this is suppose to happen, and I'm not finding anything on it, so I thought I would mention it.
You are unable to use the customize feature on home and group landing pages, and the admin filter menu is disabled as well as other JavaScript features.
I tested it on firefox, but forgot to test on others.
Comment | File | Size | Author |
---|---|---|---|
#17 | commons-make_build_problems_with_timeago-1826586-17.patch | 894 bytes | dnotes |
#16 | commons-make_build_problems_with_timeago-1826586.patch | 1.22 KB | dnotes |
#12 | commons-timeago-info-1826586-6685482.patch | 739 bytes | Topcheese |
Comments
Comment #1
ezra-g CreditAttribution: ezra-g commentedAre you showing any errors in the Javascript error console?
Comment #2
geigerg CreditAttribution: geigerg commentedSCRIPT5007: Die Eigenschaft "settings" eines undefinierten oder Nullverweises kann nicht abgerufen werden.
timeago.js, Zeile 13 Zeichen 1
SCRIPT438: Das Objekt unterstützt die Eigenschaft oder Methode "timeago" nicht
timeago.js, Zeile 3 Zeichen 5
SCRIPT5007: "settings" is undefined or null
timeago.js, line 13 character 1
SCRIPT438: Object does not support the property or method "timeago"
timeago.js, line 3 character 5
Comment #3
gnugetThe problem is the jquery.timeago.js file.
seems to the drush make file fails when try to download the jquery.timeago.js plugin.
this is what contains the jquery.timeago.js file after to run drush make:
In the meanwhile you can download the file from here:
http://timeago.yarp.com/jquery.timeago.js
And put the file in
profiles/commons/modules/contrib/timeago
And now all should work fine.
Comment #4
geigerg CreditAttribution: geigerg commentedDoesn't work.
Comment #5
ezra-g CreditAttribution: ezra-g commentedThanks for the troubleshooting, gnuget!
I switched us to downloading the master version of the library from github and it appears to download with the right file contents.
http://drupalcode.org/project/commons.git/commit/751bd3b
I re-opened #1821996: Please include Time Ago library in whitelist to allow us to use a specific commit hash from github so that we can use a known version of the library in the long-term.
Comment #6
ezra-g CreditAttribution: ezra-g commentedComment #7
ezra-g CreditAttribution: ezra-g commented@geigerg - Did you clear your site cache and shift+refresh your browser?
Comment #8
geigerg CreditAttribution: geigerg commentedI have replaced the file before setup - still the same error as on #2.
Comment #9
gnuget@geigerg
Make sure to delete the old jquery.timeago.js before to download the new one.
I'm sure with this should work
Comment #10
geigerg CreditAttribution: geigerg commentedLatest dev - still the same problem.
Here the firebug output.
TypeError: jQuery.timeago is undefined
[Break On This Error]
jQuery.timeago.settings.strings = {
timeago.js?mcwqgx (line 13)
TypeError: jQuery("abbr.timeago, span.timeago, time.timeago", context).timeago is not a function
[Break On This Error]
jQuery('abbr.timeago, span.timeago, time.timeago', context).timeago();
Comment #11
Topcheese CreditAttribution: Topcheese commentedI haven't had a chance to test it yet, but I did grab the latest code to look at. I'm trying my best to understand Drupal, but if I had to guess the problem is the the timeago.js file is not included in the timeago.info file.
scripts[] = timeago.js
Comment #12
Topcheese CreditAttribution: Topcheese commentedI'm sure this is not needed, but I was anxious to throw my first patch out and get the process down.
Edit: I corrected a typo is also why I posted the patch just in case it was missed.
Edit: Actually I think it might depend on comment #5
Comment #13
gnugetTopcheese
The js file is not added at the info file because is not part of the module, that plug in needs to be downloaded separately.
So, the bug is related with drush make because fails when download the js file and not with the module in self, Geirger you are doing something wrong I think, because this should work, can you double check if the file is in the correct place?
Comment #14
gnugetComment #15
geigerg CreditAttribution: geigerg commentedChecked it again - latest dev (2012-02-11), Setup on Aquira Dev Desktop
As long libraries api is disabled timeago and other js are working fine. After enabling libraries all js is broken.
Comment #16
dnotes CreditAttribution: dnotes commentedSo, I doubt that this will fix geigerg's problem, but re: the file not downloading, using drush 4.5 (the version from ubuntu 12.04) will not download the jquery.timeago.js unless you include the overwrite flag. See attached patch if you would like to support that. So far as I can tell it doesn't break 5 either.
Comment #17
dnotes CreditAttribution: dnotes commented...and this one passes verify-makefile.
Comment #18
geigerg CreditAttribution: geigerg commentedThis is not a downloading problem - see #15 : since #5 the jquery.timeago.js is in place and working.
After enabling the libraries module all js is broken. After disabling all js incl. jquery.timeago.js is working fine.
Comment #19
geigerg CreditAttribution: geigerg commentedMaybe this will help - I will test it later.
http://drupal.org/node/1832550
Comment #20
geigerg CreditAttribution: geigerg commentedTimeago latest DEV, moved jquery.timeago.js to /sites/all/libraries/timeago/ -> works fine.
Comment #21
Topcheese CreditAttribution: Topcheese commentedI'll have to check again, but I'm pretty sure it doesn't work correctly on comment display. Instead of the rendered output it's showing the span tag.
Comment #22
ezra-g CreditAttribution: ezra-g commentedThanks for the bug report, Topcheese. This seems unrelated to using a specific hash of the Timeago library -- I re-opened #1838928: Timeago display displays HTML on comments so we can track the incorrect formatting.
Comment #23
Topcheese CreditAttribution: Topcheese commentedNo problem ezra-g, I love doing this kind of stuff. Well, maybe I should post a problem while I'm at it. I believe we are talking about two different issues here. The original problem described above and as described in #18 still exist with the latest dev? Enabling Libraries breaks Javascript, so are there not plans to support Libraries module, and should I post a new issue?
Comment #24
ezra-g CreditAttribution: ezra-g commentedI marked #1802252: commons.profile makes an Aegir install fail as a duplicate.
Comment #25
mxmilkiib CreditAttribution: mxmilkiib commentedWith the patch, I can now build a platform on Aegir.
(it still fails creating a site, so I'll either continue in the other issue or create a new one shortly)
Comment #26
Chipie CreditAttribution: Chipie commentedIf have applied the patch in #17, but I can't create a platform. I get an error:
Directory not empty: /tmp/drush_make_tmp_1354618899_50bdd81366585/__build__/profiles/commons/modules/contrib/timeago
Any ideas?
Comment #27
Devin Carlson CreditAttribution: Devin Carlson commentedThis should now be fixed with #2012832: Timeago Library Update.