Closed (duplicate)
Project:
Drupal core
Version:
8.0.x-dev
Component:
shortcut.module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
24 Apr 2013 at 17:26 UTC
Updated:
29 Jul 2014 at 22:13 UTC
Jump to comment: Most recent file
Comments
Comment #1
xtfer commentedComment #2
dutchyodaComment #3
dutchyodaI created a seperate Elements file, so it can be used in both link add and link edit.
Furthermore I made the changes to make everything work by controller.
Comment #4
h3rj4n commentedThere should be an enter at the end (multiple).
Should be {@inheritdoc} right?
Comment #5
ajitsMade changes according to #4.
Comment #6
ajitsJust re-rolled the previous patch with the changes suggested. However, I see that the menu style for
admin/config/user-interface/shortcut/linkstill needs to be changed to new routing system.Should the status be changed to "needs work"? Or will that part be taken care of in some other issue?
Comment #7
h3rj4n commentedNo, that should be fixed in this issue. This definitely needs work! ;)
As long as the routing.yml and the hook_menu aren't changed, none of the code of the patch is tested. So that needs to be in this patch.
Comment #8
pfrenssenI updated routing.yml and hook_menu(). The access check was identical to the one used for shortcut_link_delete, so rolled them into one.
Comment #9
pfrenssenComment #11
pfrenssen#8: 1978972-8-shortcut-controller_conversion.patch queued for re-testing.
Comment #13
pfrenssenComment #15
tim.plunkettThis needs to add/change the same code as #1978966: Convert shortcut_link_add and shortcut_link_edit to a Controller, merging them.