on a brand new install I have this:

jQuery.timeago is undefined
timeago/timeago.js? Line 13

I grabbed this version of the jquery file:
https://raw.github.com/rmm5t/jquery-timeago/master/jquery.timeago.js"

timeago.js and jquery.timeago.js are being loaded into the browser.

CommentFileSizeAuthor
#4 timeago-wrong-file-include-order-1832550-4.patch564 bytesicecreamyou

Comments

socialnicheguru’s picture

socialnicheguru commented

after the standard install, the only other modules are libraries and timeago.

the timeago library is version 0.11.4

icecreamyou’s picture

icecreamyou commented
Status: Active » Postponed (maintainer needs more info)

To be clear, when you go to yoursite.com/sites/all/modules/timeago/jquery.timeago.js, the file that comes up is identical to the one at http://timeago.yarp.com/jquery.timeago.js ?

Can you check in your page source and see if jquery.timeago.js gets loaded before or after timeago.js?

socialnicheguru’s picture

socialnicheguru commented
Status: Postponed (maintainer needs more info) » Active

jquery.timeago.js loads after timeago.js

moved the file to sites/all/libraries/timeago.

I have the following:

from the source file they seem to be loaded in this order:

jQuery.extend .... "sites\/all\/modules\/apps-commons-1\/timeago\/timeago.js":1,"sites\/all\/libraries\/timeago\/jquery.timeago.js":
icecreamyou’s picture

icecreamyou commented
Status: Active » Needs review
StatusFileSize
new timeago-wrong-file-include-order-1832550-4.patch564 bytes

Does this fix it?

socialnicheguru’s picture

socialnicheguru commented
Status: Needs review » Active

fixed it!

socialnicheguru’s picture

socialnicheguru commented
Status: Active » Fixed
icecreamyou’s picture

icecreamyou commented
Status: Fixed » Reviewed & tested by the community

Good! But the patch still needs to be committed so switching this issue to RTBC

icecreamyou’s picture

icecreamyou commented
Status: Reviewed & tested by the community » Fixed

Committed fix to dev

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

jordanmagnuson’s picture

jordanmagnuson commented

Just wanted to say that this patch fixed the error for me as well. Might want to commit it to a release version of the module one of these days.

icecreamyou’s picture

icecreamyou commented

Fair enough. I just tagged new releases