The Save button is missing on the /admin/config/user-interface/shortcut/manage/default/customize page. This prevents saving reordered shortcuts links. I'm logged in as user-1 with all shortcut module permissions enabled.

Comments

mducharme’s picture

Assigned: Unassigned » mducharme
mducharme’s picture

Status: Active » Needs review
StatusFileSize
new637 bytes

The shortcut set customize form was missing the #type property in the actions function. It should be a quick review.

mgifford’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new23.3 KB

What? You want to both re-order that list AND save it? :)

Seriously though, good catch. This is a simple oversight by someone and really glad it's fixed. Thanks for the patch.

With the save button

mparker17’s picture

Assigned: mducharme » Unassigned
tim.plunkett’s picture

Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs tests

Nice work!
This needs a web test to prevent further regressions.

martin107’s picture

Status: Needs work » Needs review
StatusFileSize
new1.35 KB
new1.97 KB

Here is the outline of the WebTest, its not working yet but I am posting what I have got, in the hope, that I have done something trivially silly and someone can point out my shame in 2 seconds flat!!!

The Test:-

1) login with correct previlages
2) post to relevant page
3) look for HTML 200 status code
3) look for green message lable text on returned page

Not quite sure why it is failing .... through web browser testing it fail to return stop ... debugging fails to hit my breakpoints.
At this point I just want to see what testbot says.

martin107’s picture

Please don't be fooled by the passing tests.

the new test \Drupal\shortcut\Tests\Form\SetCustomizeFormTest does not appear in the test results.. I will look at this again soon

Although the test is run by

php core/scripts/run-tests.sh --php /Applications/MAMP/bin/php/php5.5.10/bin/php --color --die-on-fail --url http://dev.drupal.co.uk --verbose --file ./core/modules/shortcut/tests/src/Form/SetCustomizeFormTest.php

swentel’s picture

mgifford’s picture

Status: Needs review » Needs work
Issue tags: +Needs reroll
swentel’s picture

Status: Needs work » Needs review
StatusFileSize
new671 bytes

Rerolled but without tests - I would fix those in #1991104: Shortcut links can't be modified before saving a shortcut set (causes shortcuts in the toolbar to appear in alphabetical order) because there's a deeper problem with weights.

devin carlson’s picture

Issue tags: -Needs tests, -Needs reroll
StatusFileSize
new2.95 KB
new2.29 KB

Added basic test coverage for the shortcut set customize (list links) page.

Leaving testing of the shortcut order to #1991104: Shortcut links can't be modified before saving a shortcut set (causes shortcuts in the toolbar to appear in alphabetical order) per #8 and #10.

Status: Needs review » Needs work

The last submitted patch, 11: 2285245-11-tests-only.patch, failed testing.

devin carlson’s picture

Status: Needs work » Needs review
mgifford’s picture

Status: Needs review » Reviewed & tested by the community
alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed f48f760 and pushed to 8.0.x. Thanks!

  • alexpott committed f48f760 on 8.0.x
    Issue #2285245 by Devin Carlson, martin107, swentel, mducharme: Fixed...

Status: Fixed » Closed (fixed)

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