I'm not sure where this is coming from. Head to /admin to see see the button.

install.php_2.diff.png

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

LewisNyman’s picture

Title: “Install new theme” action link appear on /admin » “Install new theme” action link appears on /admin
LewisNyman’s picture

Component: system.module » update.module

This is actually related to the update module. Looking around, I guess they sit here because there is no admin/theme path?

LewisNyman’s picture

FileSize
896 bytes

I figured it out! Who said this new routing system was hard work?

LewisNyman’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, theme-action-2102357-3.patch, failed testing.

LewisNyman’s picture

Status: Needs work » Needs review

#3: theme-action-2102357-3.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, theme-action-2102357-3.patch, failed testing.

LewisNyman’s picture

Status: Needs work » Needs review
FileSize
745 bytes

I couldn't find any references to these paths that could be breaking the tests. I don't actually need to change the key so let's try that.

tim.plunkett’s picture

It doesn't help when all the test does is assert text DOESN'T show up, and then the page starts 404-ing :)

Berdir’s picture

Should we also rename the routes (as the first patch did) and move the dynamic local actions in update_menu() to yml?

tim.plunkett’s picture

I'd rather just fix the bug, and do the YAML-ification/route rename in a dedicated issue.

@LewisNyman btw, good find and nice fix.

LewisNyman’s picture

Thanks guys, nice to find time to explore other areas. Is there anything left to do in this issue?

znerol’s picture

Issue summary: View changes
Status: Needs review » Needs work

Meanwhile the button is not shown anymore on that page. However, the behavior of the theme admin page is still not consistent:

  1. Appearance - List (default tab) is on admin/appearance (Breadcrumb: Home - Administration)
  2. Appearance - Update tab is on admin/theme/update (Breadcrumb: Home - Administration)
  3. Appearance - Settings tab is on admin/appearance/settings (Breadcrumb: Home - Administration - Appearance)
  4. Install new theme button leads to admin/theme/install (Breadcrumb: Home - Administration)

Also note the path admin/appearance/install is used for the install action link placed underneath disabled themes.

LewisNyman’s picture

Status: Needs work » Fixed

@znerol That sounds like a completely different issue, so you want to open a new one?

Status: Fixed » Closed (fixed)

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