[dev] [commit] r1000 - templates/diacritics_fix wwwbase wwwbase/styles

[Cătălin Frâncu]

Uraaa, commitul 1000. :-)

Alin, tocmai m-am prins că tu nu ești pe lista dev at dexonline.ro. Am uitat eu să te adaug, sau te-ai dezabonat tu? Că acolo discutăm toate commit-urile, dăm code review etc. Am să-ți retrimit mesajele recente, sunt câteva cu feedback pentru tine.

> Added: templates/diacritics_fix/diacritics_fix.ihtml

Hai să-l punem chiar în templates/common/diacritice.ihtml, prin simetrie cu diacritice.php. De asemenea, nu ar trebui să fie HTML complet, ci strict bucata de conținut. După care din diacritice.php o apelezi cu:

SmartyWrap::displayCommonPageWithSkin('diacritice.ihtml');

> +				{$textarea}
> +				{$hiddenText}

Nu îmi este clar ce este cu $hiddenText ăsta. Am văzut că îl întreții pe parcursul codului, dar nu înțeleg de ce. Poți să documentezi printr-un comentariu, te rog?

> -		self::$nonDiacritics = pref_getSectionPreference("crawler", "non_diacritics");
> +		self::$nonLowerDiacritics = pref_getSectionPreference("crawler", "non_lower_diacritics");
> +		self::$nonUpperDiacritics = pref_getSectionPreference("crawler", "non_upper_diacritics");

Ești sigur că ai nevoie de toate astea trei? Poate te descurci cu mb_strtoupper și mb_strtolower, merge bine și cu diacritice.

>   			$this->resultText .= mb_substr($this->text, $this->lastOffset, $offset - $this->lastOffset);
> +			$this->hiddenText .= mb_substr($this->text, $this->lastOffset, $offset - $this->lastOffset);

Hai să facem din mb_substr(...) o variabilă, ca să nu duplicăm codul.

> +				$this->hiddenText .= "@@".($this->selectCount - 1)."@@";

Ce înseamnă @@ aici? E o notație a ta proprie?

> +			$this->resultText .= mb_substr($this->text, $this->lastOffset, $offset - $this->lastOffset + 1);
> +			$this->hiddenText .= mb_substr($this->text, $this->lastOffset, $offset - $this->lastOffset + 1);			

Idem ca mai sus, hai să nu duplicăm codul.

> +		if (isset($_POST['hiddenText'])) {

E de preferat să citești la începutul scriptului argumentele trimise scriptului, folosind

$hiddenText = util_getRequestParameter('hiddenText');

În felul ăsta vezi mai clar ce parametri acceptă scriptul. Iar util_getRequestParameter e un pic mai flexibil -- merge și cu GET și cu POST, acceptă valori implicite etc.

Cătălin