When looking at admin/apps/server/appname/details you get the nice title callback'd App title, however if you go to only admin/apps/server/appname (no /details) the title and breadcrumb disappear. Figure that out.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

srjosh’s picture

e2thex’s picture

So I did some investigating and the title callback is getting called, just the value it returns is not being used.

e2thex’s picture

So part of the problem had to do with the server and app pages using MENU_CALLBACK instead of MENU_NORMAL_ITEM.

But after changing those the tabs still do not seem to have breadcurmbs.

e2thex’s picture

Status: Active » Needs work
FileSize
4.67 KB

Ok I went back and work out the use of the load arguments in the paths. Change it so that we have both a %apps_server and %apps_app in the paths for the apps.

I do not fully understand why this has fixed the issue with title and breadcrumbs but it has, also it is a little more clear what we are trying to do there.

febbraro’s picture

+++ b/apps.moduleundefined
@@ -177,7 +188,7 @@ function apps_menu() {
-
+function apps_test() { return "BOB";}
 /**
  * Implements hook_theme().
  */
@@ -426,7 +437,7 @@ function apps_server_load($name) {

Remove test code.

febbraro’s picture

Here is a reroll against head. It applies cleanly but it does not fix the problems as best I can tell.

  • hefox committed d243c41 on 7.x-1.x
    Issue #1751732: Make sure the app pages link back to app listing
    
hefox’s picture

Issue summary: View changes
Status: Needs work » Fixed

I think core changed to fix this, but I think I fixed the damn breadcrumbs being totally messed up.

  • hefox committed d243c41 on 7.x-1.x-1411912
    Issue #1751732: Make sure the app pages link back to app listing
    

Status: Fixed » Closed (fixed)

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