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

juliangb’s picture

Status: Active » Needs review
FileSize
1.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.

christopher.boss’s picture

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!

juliangb’s picture

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.

juliangb’s picture

juliangb’s picture

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.

juliangb’s picture

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

Let's try this for 7.x.

Francewhoa’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
45.99 KB
48.8 KB
46.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

juliangb’s picture

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.

juliangb’s picture

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.