-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[PR#160] Big update #158
Comments
Je sens qu'elle va être lourde cette PR :) |
|
Je trouve encore quelques bugs en testant des modifications sur les addons [POC], je commit quand j'ai fini le debug et je te dirai (@BoboTiG) quand le PR sera suffisamment stable |
Upd commit
je n'ai pas fait de debug pour le coup Petit exemple pour le [POC] (3.7) :
|
J'ai foiré mon debug là, j'ai relevé quelques bugs, je prépare un fix pour cette nuit/demain matin, ne pas hésitez à mettre dans cette issue tout ce qui déconne ;) A. admin/addons ->problème de filtre qui ne s'applique pas ou retourne des erreurs :/ |
(Je mettrai à jour ce commentaire pour les autres trouvailles.)
|
Certains points un peu "dirty", je ferais le propre quand on aura valider le tout ;) |
encore quelques bugs pour : (2.) refresh du cache lors de l'activation/desactivation des addons |
Je ne crée pas de nouvelle issue pour mon dernier PR (#166), quelques points sont relatifs à cette issue ;)
|
Quand c'est bon pour toi, tu peux clôturer, histoire d'avancer ;) |
PR #160 changelog
install.php
favatar.php
create_folder()
+ test sur création du dossier de stockage$targetDir
aprés les tests qui peuvent conduire à unexit()
(pas de casse constaté) (micro optimisation)petits fix de typo à droite et à gauche
define
/.cache/
->/var/cache/
[vhost ready]/var/{vhost}/
[vhost ready]/var/{vhost}/addons/
[vhost ready]/var/{vhost}/cache/
[vhost ready]error log
add d'un POC dans
/inc/boot.php
pour un systéme de log des erreurs viaini_set('error_log',...)
et l'ajout d'une nouvelle variable globale$GLOBALS['errors']
je suis pas très convaincu par la méthode :
/var/php-error.log
, path à définir/var/log/
? ou par vhost/var/{vhost}/log/
?php-error.log
->YYYY-MM-DD-php-error.log
Addons [vhost ready] (core)
.ini
->.php
version compliancy
cf.\inc\addons.php
->addon_test_compliancy()
à valider.$declaration
via la fonctionaddon_test_declaration()
\admin\inc\addons.php
->\inc\addons.php
(marre de jongler entre 2 fichiers pour le refactor/debug des addons)\admin\inc\addons.php
pour tester une présentation différente (il me semble que @BoboTiG avait lancé l'idée...)DIR_VHOST_ADDONS/{addon_id}/cache/
->DIR_VHOST_CACHE/addon-{$addon_id}/
, ça permet de regrouper tout les caches au sein du dossierDIR_VHOST_CACHE
et ça réduit le nombre de niveau du pathaddon_get_translation()
ajout d'un fallback vers la langue ENaddon_form_edit_settings()
to useaddon_get_translation()
ADDONS_DB upd pour
/databases/addons.php
->/var/{vhost}/databases/addons.php
[vhost ready]feeds
cache_rss_array.dat
->cache1_feed.dat
c_{rss|atom}_{random}.dat
->cache2_{rss|atom}_{random}.dat
theme defaut
?format={atom|rss}
-> retour à rss.php et atom.php (dans le themes defaut pas dans les scripts de flux)class="{tpl_class}"
dans<body>
HTTP header
http_response_code(404)
dansindex.php
, à vérifier qu'il ne manque pas des HTTP 404 sur certains cas, et si pas de faux positifhttp_response_code()
qui peux servir de fallback, je la mets au cas où... Si pas utilisée d'ici la freeze 3.7, on vire...refactor
/admin/inc/sqli.php
/inc/sqli.php
to/admin/inc/sqli.php
langs files
/inc/lang/en-en.php
en comparant les fichiers langscreate_folder
folder_secure_parents()
qui parcourt tous les enfants d'un path pour mettre en place unindex.php
et éventuellement un.htaccess
create_folder()
pour utiliserfolder_secure_parents()
j'en oublie
vhosts
core
$GLOBALS['racine']
parURL_VAR
(url du blog) sauf dans ce qui touche la partie formulaire des preferences/settingsBoboTiG/blogotext-addons (à venir, lundi)
VALIAS [POC] (vu en privé avec/par @BoboTiG (serait bien une issue ;))
VALIAS ? ça vous permet de mettre plusieurs URL/domain/sous domaine sur le même vhost et d'afficher les mêmes théme, image, article ...
exemple :
blog-1.fr
etblog-2.fr
sur le même vhostC'est un POC, il s'agit de démontrer la faisabilité, il n'y a pas de gestion des addons, des préférences (...)
idem pour la partie admin, préférez passer par le vhost.
Au cas où le POC merdouille, supprimer (ou renommer)
/var/blog-2.fr/settings/valias.php
.je n'ai pas fait de debug pour le coup
Petit exemple pour le [POC] (3.7) :
http://blog-1.fr/
/var/blog-1.fr/
/var/localhost-blog-2/settings/valias.php
/var/blog-2.fr/settings/valias.php
pour y enregistrer :/var/blog-2.fr/settings/valias.php
le blog 2 ne sera plus accessiblefolder_secure_childs()
>folder_secure_parents()
commit
The text was updated successfully, but these errors were encountered: