#5833 closed defect (bug) (fixed)
What's new activity update failing on undefined js var
| Reported by: | hnla | Owned by: | imath |
|---|---|---|---|
| Priority: | normal | Milestone: | 2.1 |
| Component: | Activity | Version: | |
| Severity: | major | Keywords: | dev-feedback |
| Cc: | hnla |
Description
Attempting to post an activity update is failing due to var ltext being undefined.
We appear to have re-factored a section of js for activity stream updates lines: 184 through to 190.
line 190 if ( ltext !== '' ) {
was != and it was declared earlier as var ltext = jq('#activity-stream li.new-upd...
This now hangs on undefined variable after submitting the update.
Strict type comparison or 'null' doesn't change matters so to get running I've declared the variable before L:184 if block.
Change History (3)
#3
@
12 years ago
Hmm interesting there's a discrepancy in files from my end, as that line termination error didn't show in the file updated from trunk it was correctly stated as ';', yet when I do a 'Diff' it does show, it's to do with fact I incorrectly used the src dir when setting up the new path to checkout from but I'm puzzled as to how this new format an grunt stuff does work.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Hi @hnla,
Thanks for your feedback, I think it's due to the fact there's a '.' instead of a ',' at line 186.
As a result, I confirm that posting an activity from the user's profile generates the js error you mentioned.
I'm going to commit, it should fix the issue.