[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