[dev] [commit] r912 - patches phplib phplib/models templates/admin tools wwwbase/admin wwwbase/elfinder-connector wwwbase/js wwwbase/styles

Catalin Francu cata at francu.com
Mon Jul 29 15:29:03 EEST 2013


A venit băiatu' :-) Încep de la cele mai vechi. Fac îndată un commit în
care rezolv toate aceste comentarii, mai puțin ultimul, marcat cu ****, pe
care ți-l las ție.

visualTag.js: trebuie [20], nu de [18], altfel cele două JS-uri nu pot
coexista.

elFinderLogger.class.php: Pare luată ca atare, fără modificări, nu? De
acord.

Visual.php: escapeSlashes ar trebui să nu fie necesară, căci cam sună a
mânăreală. Am găsit o
pagină<https://github.com/Studio-42/elFinder/issues/281> care
spune că ar avea legătură cu magic
quotes<http://php.net/manual/en/security.magicquotes.php>.
Oricum, ar sta mai bine în phplib/StringUtil.php.

Update: Am investigat un pic și m-am prins: poți folosi str_replace în log
de preg_replace, căci nu este nevoie de expresii regulate. Și atunci
slashurile nu mai sunt speciale.

Toate apelurile de tip where_like('path', "{$path}") pot fi înlocuite cu
where() simplu, căci nu există caractere de regexp. De aici, putem folosi
mai departe un sintactic sugar pe care l-am scris eu,
Visual::get_by_path($path). Merge cu orice nume de câmp, vezi
phplib/models/BaseObject::__callStatic().

**** Comanda "duplicate" nu pare să aibă efect în baza de date (am dat
click-dreapta pe o poză și am selectat "duplicate"). Nu merge nici la
directoare, fie din lista de thumbs, fie din meniul din stânga.

Cătălin


On Thu, Jul 25, 2013 at 2:47 PM, <automailer at dexonline.ro> wrote:

> Author: grigoroiualex
> Date: Thu Jul 25 14:47:04 2013
> New Revision: 912
>
> Log:
> Fully functional elFinder file manager for Visual Dictionary. TODO: add
> PRIV_VISUAL to permissions list
>
> Added:
>    patches/00089.sql
>    phplib/models/Visual.php
>    templates/admin/definitionImages.ihtml
>    wwwbase/admin/definitionImages.php
>    wwwbase/elfinder-connector/elFinderLogger.class.php
>    wwwbase/elfinder-connector/elFinderModToDB.class.php
>    wwwbase/elfinder-connector/visual_connector.php
> Deleted:
>    wwwbase/elfinder-connector/connector.php
> Modified:
>    phplib/SmartyWrap.php
>    tools/setup.sh
>    wwwbase/elfinder-connector/wotd_connector.php
>    wwwbase/js/jquery.Jcrop.min.js
>    wwwbase/styles/jquery.Jcrop.min.css
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://list.dexonline.ro/pipermail/dev/attachments/20130729/1593b14c/attachment.html>


More information about the Dev mailing list