View Issue Details

IDProjectCategoryView StatusLast Update
0003057OXID eShop (all versions)1.06. Search, Tagspublic2012-12-07 14:39
Reportertjungcl Assigned To 
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionno change required 
Product Version4.5.0 revision 34568 
Fixed in Version4.5.1 revision 38045 
Summary0003057: search default is uppercase forced in tpl
DescriptionThe value for the default search value is ident SEARCH_TITLE from lang-php.
It is forced to uppercase in the template (with |upper). This should better be done via CSS or in the lang-php directly because as its now the tpl has to be changed if a shopowner wants the text not uppercased.
TagsSearch
ThemeAzure
BrowserAll
PHP Versionany
Database Versionany

Activities

Linas Kukulskis

2011-07-14 16:50

reporter   ~0004861

in team we decided that |upper is the best way to do that, because only with css its impossible to do (entered users text must be as typed not converted). to do it in language file also not very good.

tjungcl

2011-07-15 09:49

reporter   ~0004862

Ok. The css way doesnt work, your right.

So I removed the |upper from my lokal copy of search.tpl
and that solved the bug https://bugs.oxid-esales.com/view.php?id=3056

The sDefaultValue for the function oxInnerLabel has to be |upper too, or the upper has to be removed alltogether and the SEARCH_TITLE in lang.php just be made uppercase.

You wrote thats not very good. Why?
If a user wants it uppercase, he can make in uppercase, if not then not. Whats wrong about it?

Linas Kukulskis

2011-07-15 10:38

reporter   ~0004864

0003056 already fixed.
use uppers in language files is not proper, because all text styling we do with css or with smarty modifiers and only in last way is to uppercase in lang file.