Reference: https://www.drupal.org/core/beta-changes
Issue category Task because we should update all third party assests/libraries upto date.
Issue priority Normal

Problem/Motivation

Backbone Current version 1.1.2
New version 1.2.1

http://backbonejs.org/#changelog
https://github.com/jashkenas/backbone/

Proposed resolution

Patch attached

Remaining tasks

Needs Reviewers eyes.

User interface changes

API changes

Beta phase evaluation

Comments

tohesi’s picture

Assigned: tohesi » Unassigned
Status: Active » Needs review
StatusFileSize
new103.74 KB
droplet’s picture

Status: Needs review » Needs work

Big changes. Applying patch, top right edit button disappeared

nod_’s picture

Yeah their changelog is massive it's been more than a year since their last realease: http://backbonejs.org/#changelog

Too bad jessebeach is not around anymore.

It's a novice issue in the sense that a backbone dev should find it relatively easy to fix.

droplet’s picture

Status: Needs work » Needs review
StatusFileSize
new106.21 KB

this issue: https://github.com/jashkenas/backbone/issues/3611

Only 2 places used this pattern, we can do code refactoring before upstream fixing. It's even more readability.

nod_’s picture

Need to add the unminified backbone source too (for the map file).

nod_’s picture

Status: Needs review » Needs work
chananapeeyush’s picture

Issue summary: View changes
Status: Needs work » Needs review
StatusFileSize
new71.23 KB
new177.44 KB

Updated issue summary.

tohesi’s picture

Title: Update Backbone to 1.2.0 » Update Backbone to 1.2.1
Issue summary: View changes
StatusFileSize
new177.4 KB
new110.25 KB

Version 1.2.1 already out. Updated files, issue title & summary accordingly.

Status: Needs review » Needs work

The last submitted patch, 8: backbone-upgrade-2494063-8.patch, failed testing.

tohesi’s picture

StatusFileSize
new177.4 KB
new1.05 KB

Forgot to update the test.

tohesi’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 10: backbone-upgrade-2494063-10.patch, failed testing.

tohesi’s picture

Status: Needs work » Needs review
nod_’s picture

Status: Needs review » Reviewed & tested by the community

Contextual links are working, toolbar, quickedit, ckeditor and the tour module as well.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 10: backbone-upgrade-2494063-10.patch, failed testing.

naveenvalecha’s picture

Status: Needs work » Reviewed & tested by the community

Setting RTBC :) As already by @nod_, Also Tested patch locally,Ckeditor, toolbar is working fine :)

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 10: backbone-upgrade-2494063-10.patch, failed testing.

dcmul’s picture

@nod_ should we not look into the reasons why the tests are failing before RTBC? or is this an exception.

dcmul’s picture

Issue tags: +Needs reroll

patch failing

dcmul’s picture

Status: Needs work » Needs review
StatusFileSize
new177.41 KB

I have done a re-roll.

daffie’s picture

Status: Needs review » Needs work
Issue tags: -Needs reroll

@dcmul: Can you tell me what you changed in the reroll. If you are able to please add an interdiff.txt.

diff --git a/core/assets/vendor/backbone/backbone.js b/core/assets/vendor/backbone/backbone.js
new file mode 100644

We do not need this file. We have backbone-min.js and backbone-min.map for that.

nod_’s picture

Status: Needs work » Needs review

No we need it, the map file is useless without the original source. Needs to be in the patch.

daffie’s picture

Status: Needs review » Reviewed & tested by the community

OK I nod_ says we need the file then we need the file. To RTBC.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 22: backbone-upgrade-2494063-22.patch, failed testing.

Status: Needs work » Needs review
daffie’s picture

Status: Needs review » Reviewed & tested by the community

Back to RTBC.

  • webchick committed 79c2dc4 on 8.0.x
    Issue #2494063 by tohesi, trwad, dcmul, droplet, nod_, naveenvalecha:...
webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.0.x. Thanks!

Thanks also for the manual testing. Always spooky when updating these JS libs. :)

Status: Fixed » Closed (fixed)

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