There should be no variables in the translatable string. Please use the placeholders to make your strings more secure and translatable.

 78:   $objects["$alias->path"] = t("View page displays having a path of '$alias->path'");
108: '#title' => t("View $alias->view_name, display $alias->display_name, on path $alias->path, with $voc->name arguments."),
CommentFileSizeAuthor
#5 1195042-3_bad_usage_of_t.patch7.19 KBemackn
#2 1195042-2_bad_usage_of_t.patch14.67 KBwojtha
#1 1195042-1_bad_usage_of_t.patch14.67 KBwojtha

Comments

wojtha’s picture

wojtha commented
Status: Active » Needs review
StatusFileSize
new 1195042-1_bad_usage_of_t.patch14.67 KB

Patch includes also a dozen of whitespace fixes.

wojtha’s picture

wojtha commented
StatusFileSize
new 1195042-2_bad_usage_of_t.patch14.67 KB

Almost same as #1, changed just one line - more appropriate placeholders.

- '#title' => t('View @view, display @display, on path @path, with @vocabulary arguments.', array('@view' => $alias->view_name,...
+ '#title' => t('View %view, display %display, on path %path, with %vocabulary arguments.', array('%view' => $alias->view_name,...
wojtha’s picture

wojtha commented
Version: 6.x-2.2 » 6.x-2.x-dev
emackn’s picture

emackn commented
Version: 6.x-2.x-dev » 6.x-2.2
StatusFileSize
new 1195042-3_bad_usage_of_t.patch7.19 KB

i rerolled this, seemed to be some issues applying it to dev. I'm applying it to 2.x

emackn’s picture

emackn commented
Status: Needs review » Reviewed & tested by the community
john franklin’s picture

john franklin commented
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

It looks like this patch has already been applied.

Status: Fixed » Closed (fixed)

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