Format Special Words

T is only meant for translation of strings. It's not meant to be used for capitalization needs. Instead, there's currently code for capitalization that's been stubbed out in the module. But, it hasn't been implemented yet. The basic idea is that you can set the strings to capitalize on the crumb configuration screen, then those should be used when building the crumbs.

This is still an open todo.

Ahh. Sorry, I didn't realize you were on the old version.

The 7.x branch already has the Capitalizor implemented.

The difficulty with applying t() is that it would make duplicate strings available to translators which is already super confusing as it is in D7. You could do that with a patch in your local project. But, we can't do that in the contrib version. Instead, we could expand the capitalizer to support full word caps. Development is basically frozen on the D7 branch. So, it's unlikely that any patches you make will be broken in the future.

I'm happy to assist if someone wants to pick up support of the 7.x code base again. D7 will last 4-5 more years. So, it's worth doing.

Hi guys,

@Prizem - the main purpose of t() is to make strings translatable that are not variables. https://api.drupal.org/api/drupal/includes%21bootstrap.inc/function/t/7.x I would suggest to use format_string() instead. But let me turn back here in a few days with a patch, ok? :)

@Greg - i just started to take a look on the issues here, hope it's useful for you and for others :)

Cheers,
Balazs.

Hi all,
sorry for the long delay, but now i'd finally like to show you the patch that solves the need, described by @Prizem.
Let me ask some review on it :)
Cheers,
Balazs.

Looks good! Can you detail a test case so I can be sure I'm testing this code well?

Hi @Greg,
here is a test case that i used:
Create a new content (basic page) and set the url: once-a-time/brave-new-world
Then change the easy-breadcrumb settings and try all of them with checking the content's page where you make the block visible.
The following choices you have under 'Transformation mode for the segments' titles' select:
- None (didn't change)
- Capitalize the first letter of each word in the segment (didn't change)
- Only capitalize the first letter of each segment (didn't change)
- Capitalize all the letters of each word in the segment (new option) -> if you choose this one, the breadcrumb should be: Home >> ONCE A TIME >> Brave new world (where my content's title is 'Brave new world')
- Capitalize only the words that are set below (new option) -> If you choose this one, the ignore textarea has to disappear and a new one will be shown called 'Words to be forced to capitalized by the 'capitalizator'' where put time word to make only that one capitalize. Below the textarea, there has to be a checkbox saying 'Make the first letters of each segment capitalized.' By default it's checked. If you used these settings, the breadcrumb should be: Home >> Once a TIME >> Brave new world (if you uncheck the checkbox: Home >> once a TIME >> Brave new world)
I hope it solves the original need :)
Cheers,
Balazs.

Hi @Prizem,
thanks for your feedback!
As now i'm not close to my computer, i'll fix the mentioned issue in Wednesday and post the new patch here!
Could you give me an example how should i imagine the case sensitivity that could make you even more satisfied? :)
Cheers,
Balazs.

Hi @Prizem,
i hope i understood your request as you meant :)
Here is a fix for the default status of the checkbox and also has the improvement for the case sensitivity.
How i tested (hopefully it helps to make it RTBC)
- Create a content with the following url: once-a-Time-in-DRUpal/brave-new-world (my content's title is Brave new world)
- Test the following settings of easy_breadcrumb when you choose 'Capitalize only the words that are set below' option:
Set the words in "Words to be forced to capitalized by the 'capitalizator'" textarea: druPAL Time
When you check 'Use case sensitivity when matching words to be forced to capitalization.' checkbox, the result should be: once a TIME in DRUpal (if you make the second checkbox also checked: Once a TIME in DRUpal) in the breadcrumb block.
When you uncheck 'Use case sensitivity when matching words to be forced to capitalization.' checkbox, the result should be: once a TIME in DRUPAL (if you make the second checkbox also checked: Once a TIME in DRUPAL) in the breadcrumb block.

Let me know if i missed something or it doesn't work as you expect!

Bests,
Balazs.

I've rerolled the last patch on the new 7.x-2.x branch.
Waiting for review :)