Posted by wojtha on June 21, 2011 at 10:24am
2 followers
| Project: | View Alias |
| Version: | 6.x-2.2 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | reviewed & tested by the community |
Issue Summary
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."),
Comments
#1
Patch includes also a dozen of whitespace fixes.
#2
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,...
#3
#5
i rerolled this, seemed to be some issues applying it to dev. I'm applying it to 2.x
#6