Post by loadaverage » Wed Dec 02, 2009 3:06 am

dobry den panove,

predpokladam ze nekdo uz narazil na tohlensto:
http://code.google.com/p/opencart/issues/detail?id=111

opencart uklada texty ne jako utf8, ale html encoded strings.
to znamena, ze napr namiesto mena kategorie

Code: Select all

"Fólie"
je do databazy ulozeno

Code: Select all

"Fólie"
a tym padom 32 znaku limit je v skutocnosti ovela menej.
navrhol som aj riesenie v tom bug-reporte ale nas boss je
trochu rychli v zatvarani bug reportov ktore sa mu nepacia...

we is experts(tm)


New member

Posts

Joined
Fri Nov 20, 2009 2:10 am

Post by Pot'a » Wed Dec 02, 2009 3:14 am

loadaverage wrote:dobry den panove,

predpokladam ze nekdo uz narazil na tohlensto:
http://code.google.com/p/opencart/issues/detail?id=111

opencart uklada texty ne jako utf8, ale html encoded strings.
to znamena, ze napr namiesto mena kategorie

Code: Select all

"Fólie"
je do databazy ulozeno

Code: Select all

"Fólie"
a tym padom 32 znaku limit je v skutocnosti ovela menej.
navrhol som aj riesenie v tom bug-reporte ale nas boss je
trochu rychli v zatvarani bug reportov ktore sa mu nepacia...
Ahoj, o tomto nedostatku všichni víme.

Prostuduj toto vlákno.

Čeština (v1.3.x, v1.4.x) | IČ a DIČ zákazníka (v1.3.x, v1.4.x) | XML feed zbozi.cz


Active Member

Posts

Joined
Tue Sep 01, 2009 1:48 pm
Location - Prague [CZE]

Post by loadaverage » Wed Dec 02, 2009 5:29 am

diky.

proc davate vsechno do jedneho threadu?..
jasne ze to nikdo nenajde :]

uz to niekto daval ako bug report okrem mna?
v tom vlaknu tvrdite ze
Danielův zápis funkce htmlentities() má svůj důvod
ja si s tym dovolim nesuhlasit. kedze je tam hned konverzia do utf8,
ziadny takyto dovod nevidim a vyfiltrovat html special chars je
podla mna uplne dostacujuce. ci nie?

ale diky za upozorneni v fckeditor, tam by ma tiez nenapadlo ze
niekto zapne konvertovanie na html entities. ved nie sme v '98 proboha...

we is experts(tm)


New member

Posts

Joined
Fri Nov 20, 2009 2:10 am

Post by loadaverage » Wed Dec 02, 2009 5:39 am

btw. v novej verzii ckeditor vypnut entities:

Code: Select all

admin/view/javascript/ckeditor/config.js:
        config.entities = false;
este raz diky za heads up.

we is experts(tm)


New member

Posts

Joined
Fri Nov 20, 2009 2:10 am

Post by Pot'a » Wed Dec 02, 2009 5:44 pm

loadaverage wrote:diky.

proc davate vsechno do jedneho threadu?..
jasne ze to nikdo nenajde :]
To tak nějak samo vyplynulo :)
loadaverage wrote:uz to niekto daval ako bug report okrem mna?
v tom vlaknu tvrdite ze
Danielův zápis funkce htmlentities() má svůj důvod
ja si s tym dovolim nesuhlasit. kedze je tam hned konverzia do utf8,
ziadny takyto dovod nevidim a vyfiltrovat html special chars je
podla mna uplne dostacujuce. ci nie?

ale diky za upozorneni v fckeditor, tam by ma tiez nenapadlo ze
niekto zapne konvertovanie na html entities. ved nie sme v '98 proboha...
Mno, podle mě jde o možná paranoidní zabezpečení, Danielovi se patrně nechce používat htmlspecialchars() a regex pro formulářová pole.

Já osobně žádnout filtraci nepoužívám, tak snad mě nic zlého nečeká :)

Myslím, že nikdo kromě tebe do issue track nic nereportoval. FCKeditor jsem nahradil WYMeditorem.

Čeština (v1.3.x, v1.4.x) | IČ a DIČ zákazníka (v1.3.x, v1.4.x) | XML feed zbozi.cz


Active Member

Posts

Joined
Tue Sep 01, 2009 1:48 pm
Location - Prague [CZE]

Post by loadaverage » Thu Dec 03, 2009 12:18 am

pouzivat html entities v tomhle pripade je absolutnej nonsense.
presne na toto je htmlspecialchars.

ale kdyz uz sel touto cestou, tak by musel vsude decodovat entities
pred tym nez kontroluje dlzku textov...

we is experts(tm)


New member

Posts

Joined
Fri Nov 20, 2009 2:10 am

Post by Pot'a » Thu Dec 03, 2009 12:20 am

loadaverage wrote: ale kdyz uz sel touto cestou, tak by musel vsude decodovat entities
pred tym nez kontroluje dlzku textov...
Ano, souhlasím.

Čeština (v1.3.x, v1.4.x) | IČ a DIČ zákazníka (v1.3.x, v1.4.x) | XML feed zbozi.cz


Active Member

Posts

Joined
Tue Sep 01, 2009 1:48 pm
Location - Prague [CZE]

Post by vojtech25 » Sat Jan 30, 2010 12:12 am

loadaverage wrote:pouzivat html entities v tomhle pripade je absolutnej nonsense.
presne na toto je htmlspecialchars.
je pravda, ze pouziti html entities v tomhle pripade neni zrovna idealni, ale naky chyby se do takle velkyho systemu proste vloudej. Taky si myslim, ze misto toho vecnyho kritizovani by bylo prinosnejsi, kdybys zacal ty chyby opravovat, a byl rad, ze nekdo se ve svym volnym case zabejva tim, ze tvori tyhle opensourci... a koneckoncu nikdo te taky nenuti abys opencart pouzival, kdyz ti prijde natolik nedokonalej...

______________________________________________________
http://www.vojtechzahorsky.com


New member

Posts

Joined
Tue Jan 19, 2010 7:18 pm
Who is online

Users browsing this forum: No registered users and 4 guests