Merge lp:~openerp-dev/openobject-addons/trunk-bug-1077130-Jaydeep into lp:openobject-addons

Proposed by Jaydeep Barot(OpenERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openobject-addons/trunk-bug-1077130-Jaydeep
Merge into: lp:openobject-addons
Diff against target: 13 lines (+2/-1)
1 file modified
web_shortcuts/static/src/js/web_shortcuts.js (+2/-1)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/trunk-bug-1077130-Jaydeep
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+169368@code.launchpad.net

Description of the change

Hello,

7.0 Bookmarks must be sorted alpahbetically respecting language setting
To be change on js file in web_shotcuts

Thanks,
Jaydeep Barot.

To post a comment you must log in.
Revision history for this message
Jaydeep Barot(OpenERP) (jaydeep.barot-openerp) wrote :

Hello,

I have Fixed in https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-1077130-Jaydeep
Revision ID: <email address hidden>
revision no :- 8660

Thanks,
Jaydeep Barot.

Unmerged revisions

8660. By Jaydeep Barot(OpenERP)

[FIX] bookmarks - sort missing To Apply sortby method in js

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'web_shortcuts/static/src/js/web_shortcuts.js'
2--- web_shortcuts/static/src/js/web_shortcuts.js 2012-12-07 11:31:53 +0000
3+++ web_shortcuts/static/src/js/web_shortcuts.js 2013-06-14 09:05:46 +0000
4@@ -51,7 +51,8 @@
5 'ir.ui.menu',
6 instance.web.pyeval.eval('context', {})
7 ]).done(function(shortcuts) {
8- _.each(shortcuts, function(sc) {
9+ var sorted_sc =_.sortBy(shortcuts,function(obj){return obj.name});
10+ _.each(sorted_sc, function(sc) {
11 self.trigger('display', sc);
12 });
13 });

Subscribers

People subscribed via source and target branches

to all changes: