Hello All,

I saw some other discussions (http://drupal.org/node/744694, http://groups.drupal.org/node/112924) regarding using the String Overrides module to rename the Storm dashboard. I have successfully renamed Storm in some areas (like the Primary Links) using String Override module, but I cannot seem to override the title of the Storm dashboard itself, or the breadcrumb. I am still new to Drupal, so it is likely I may be missing something with the way overrides work.

I can rename Storm the 'wrong' way, i.e.:

function storm_storm_dashboard_types() {
  return array(
    'page' => array(
      'url'         => 'storm',
      'title'       => 'TAR dashboard',

(title is set to TAR instead of Storm)

But, this is just a bad habit to customize module code! Is there a way to rename Storm dashboard using String Override module?

Just FYI, I have tried this in settings.php:

$conf['locale_custom_strings_en'] = array(
   'Storm dashboard' => 'TAR dashboard',
);

With no luck.

Thanks so much for any help you can give!

Files: 
CommentFileSizeAuthor
#9 before.png46.1 KBFrancewhoa
#9 after.png48.8 KBFrancewhoa
#9 conf.png45.99 KBFrancewhoa
#8 storm-translation.patch1.61 KBjuliangb
PASSED: [[SimpleTest]]: [MySQL] 326 pass(es).
[ View ]
#1 storm-dashboard-string-overrides.patch1.31 KBjuliangb
FAILED: [[SimpleTest]]: [MySQL] 2,423 pass(es), 203 fail(s), and 622 exception(s).
[ View ]

Comments

Status:Active» Needs review
StatusFileSize
new1.31 KB
FAILED: [[SimpleTest]]: [MySQL] 2,423 pass(es), 203 fail(s), and 622 exception(s).
[ View ]

You should be able to do the string overrides, and I think it is a bug if you can't.

Can you try this patch? (And then your overrides)

Status:Needs review» Needs work

The last submitted patch, storm-dashboard-string-overrides.patch, failed testing.

Status:Needs work» Fixed

Hello Julian,

Wow, I appreciate your quick response!

The patch did resolve my issue with String Overrides module.

Looks like everything is working now.

Thank you so much!

Version:6.x-2.1» 6.x-2.x-dev
Category:support» bug
Status:Fixed» Needs review

Thanks for the feedback.

Just before we close this issue - I'll make sure that the automated tests pass, and we'll commit this so that it appears by default in the next download.

Title:Can I String Override instead of hacking the code?Allow dashboard title to use string overrides

Status:Needs review» Needs work

The last submitted patch, storm-dashboard-string-overrides.patch, failed testing.

Version:6.x-2.x-dev» 7.x-1.x-dev
Status:Needs work» Needs review
StatusFileSize
new1.61 KB
PASSED: [[SimpleTest]]: [MySQL] 326 pass(es).
[ View ]

Let's try this for 7.x.

Status:Needs review» Reviewed & tested by the community
StatusFileSize
new45.99 KB
new48.8 KB
new46.1 KB

juliangb, confirming this works. Thanks :) This is useful.

Using
* stringoverrides 7.x-1.8
* Fresh Drupal 7.18
* Storm 7.x-1.x-dev 2012-Dec-24
* Views 7.x-3.5
* CTool 7.x-1.2

Version:7.x-1.x-dev» 6.x-2.x-dev
Status:Reviewed & tested by the community» Needs work

Thanks for testing @Francewhoa. Committed and pushed in to 7.x-1.x.

Moving this back to 6.x-2.x, where I think the patch works, but it seems that the 6.x-2.x tests aren't passing at the moment. This will need some work to fix before we can commit other patches.

Status:Needs work» Fixed

Committed this to 6.x-2.x.

Status:Fixed» Closed (fixed)

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