Problem/Motivation

Currently links saved as link_id: link_id (Refer screenshot below). This is related to #1928082: Make usage of book.settings:allowed_types consistent
1964588-3.png

Proposed resolution

Fix the way saving links. So saved as "- link_id".

Files: 
CommentFileSizeAuthor
#12 shortcut-links.png16.16 KBheddn
#11 Selection_015.png10.33 KBTim Bozeman
#10 1971456-10.patch1.6 KBswentel
PASSED: [[SimpleTest]]: [MySQL] 58,778 pass(es). View
#7 1971456_7.patch1.58 KBTim Bozeman
PASSED: [[SimpleTest]]: [MySQL] 58,398 pass(es). View
#6 tada.png68.38 KBTim Bozeman
#6 1971456_6.patch1.62 KBTim Bozeman
PASSED: [[SimpleTest]]: [MySQL] 58,737 pass(es). View
#3 1971456-shortcut-link-3.patch3.74 KBdeveshpal
PASSED: [[SimpleTest]]: [MySQL] 58,470 pass(es). View
#1 1971456-shortcut-link-1.patch1.27 KBvijaycs85
PASSED: [[SimpleTest]]: [MySQL] 54,443 pass(es). View

Comments

vijaycs85’s picture

Status: Active » Needs review
FileSize
1.27 KB
PASSED: [[SimpleTest]]: [MySQL] 54,443 pass(es). View

Issuing patch...

jair’s picture

Issue tags: +Needs reroll
deveshpal’s picture

FileSize
3.74 KB
PASSED: [[SimpleTest]]: [MySQL] 58,470 pass(es). View

Rerolled the patch and attached the updated patch file..

Cottser’s picture

Status: Needs review » Needs work

Hm, I think this code must have moved - the rerolled patch adds an entirely new class. Needs another look.

Tim Bozeman’s picture

Assigned: Unassigned » Tim Bozeman

I shall try.

Tim Bozeman’s picture

Assigned: Tim Bozeman » Unassigned
Status: Needs work » Needs review
Issue tags: -Needs reroll
FileSize
1.62 KB
PASSED: [[SimpleTest]]: [MySQL] 58,737 pass(es). View
68.38 KB

Tada!
tada.png
I rerolled the patch from #1 because the preSave function moved from ShortcutSetStorageController.php to ShortcutSet.php

Tim Bozeman’s picture

FileSize
1.58 KB
PASSED: [[SimpleTest]]: [MySQL] 58,398 pass(es). View

Whoops. Removed a commented out line from #6.

Tim Bozeman’s picture

Stale? Patch no longer apply's.

Tim Bozeman’s picture

Status: Needs review » Needs work
swentel’s picture

Status: Needs work » Needs review
FileSize
1.6 KB
PASSED: [[SimpleTest]]: [MySQL] 58,778 pass(es). View
Tim Bozeman’s picture

FileSize
10.33 KB

Patch #10 apply's and makes the links save as - 01234567890
Selection_015.png
10++

heddn’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
16.16 KB

It works.
shortcut-links.png

amateescu’s picture

The 'links' property of shortcut sets is going away entirely in #2021779: Decouple shortcuts from menu links, would you mind postponing on that a bit?

vijaycs85’s picture

I agree with @amateescu that we are fixing something that is going to be removed in nearby future. +1 to postponed or closed(won't fix).

amateescu’s picture

Status: Reviewed & tested by the community » Postponed

Ok then, let's take the cautios step first, just in case that issue doesn't make it.

Wim Leers’s picture

Issue summary: View changes
Status: Postponed » Closed (works as designed)

This is now irrelevant indeed. Let's put this issue to rest.