Image manager - Githubissues

Jireck-npds commented 6 years ago

Hello Peux tu m'expliquer sont fonctionnement, je ne comprends pas ... Je clique sur "images manager" et la je me balade sur l'arborescence et rien ne s'affiche a part les repertoires.

Y'a un bug ou bien

previnfo commented 6 years ago

Effectivement, en sus il semble y avoir cette erreur Warning: array_key_exists() expects parameter 2 to be array, null given in ../modules/f-manager/pic-manager.php on line 86 Edit (apparemment cette notice est déjà commentée dans le fichier lui-même)

Il y a ce commentaire: // Gestion des autorisations sur les répertoires et les fichiers

/ ==> was notice on if($type f) in some case ...Coriace ..... add set type de la variable $autorise_arbo=false; controle de l'index dans le array and array_key_exists($dir, $ficlimit_fma)) et if(isset($autorise_arbo)) au lieu de if($autorise_arbo) cohérence de la correction et de ses implications encore incertaine à suivre !! <==/

Jireck-npds commented 6 years ago

Je n'ai pas la notice

previnfo commented 6 years ago

La notice je l'ai eu lors de la lecture d'un répertoire (que j'avais préalablement autorisé => picmanager)

jipexu commented 6 years ago

je ne suis pas spécialiste du file manager mais il me semble que ce comportement est normal

si il n'y a pas d'autorisation picmanager (le petit fichier) on ne voit que les répertoires
si il y a autorisation on voit les fichiers images ...

La notice je l'ai eu lors de la lecture d'un répertoire (que j'avais préalablement autorisé => picmanager)

j'ai recorrigé sur le labo pour la notice ... à retester ...

previnfo commented 6 years ago

Sur labo: (1) Dès que l'on essaye d'autoriser un repertoire (exemple: images/forum/avatars) on a cette Notice: Undefined variable: filesearch in /home/users/i/infocapagde/www/NPDS/labo/modules/f-manager/f-manager.php on line 772 (2) puis dans le répertoire autorisé (avatars) Notice: Undefined index: gene-img in /home/users/i/infocapagde/www/NPDS/labo/modules/f-manager/pic-manager.php on line 325

=> Pas d'affichage d'images dans le rep => avatars mais les images s'affichent dans d'autres rep comme "icons" lol

jipexu commented 6 years ago

(1) Dès que l'on essaye d'autoriser un repertoire (exemple: images/forum/avatars) on a cette Notice: Undefined variable: filesearch in /home/users/i/infocapagde/www/NPDS/labo/modules/f-manager/f-manager.php on line 772

fixé (il semble bien qu'il manquait un break ....)

pour la 2 ca semble beaucoup moins drôle ....

previnfo commented 6 years ago

Chez moi avec tes dernières corrections pour le (2), j'ai autant de notices (la même) qu'il y a d'images dans le rep. => pas d'affichage et cela même si je modifie les champ cache ou taille du formulaire ....

Warning: array_key_exists() expects parameter 2 to be array, null given in /home/haskonin/public_html/dune/modules/f-manager/pic-manager.php on line 86

jipexu commented 6 years ago

attend je ne comprend pas tu me parle de quel erreur ? sur le labo je sais pourquoi on a cette erreur ... (2) puis dans le répertoire autorisé (avatars) Notice: Undefined index: gene-img in /home/users/i/infocapagde/www/NPDS/labo/modules/f-manager/pic-manager.php on line 325 ...

toi tu as donc deux erreurs différentes ? ?

jipexu commented 6 years ago

Je n'ai pas la notice @Jireck-npds tu as modifier grab_globals.php ?

previnfo commented 6 years ago

Oui moi aussi je n'avais pas la même ligne commentée pour le report d'erreurs grab_globals.php Néanmoins en me mettant la même chose que labo cela ne change rien (pour moi), j'ai toujours les mêmes notices.

Petit Rectificatif: j'ai avec la précédente notice (...on line 86) bien l'affichage des images .... (il m'a fallu scrooller un moment cette répétition de même notice pour les voir en bas de page)

Jireck-npds commented 6 years ago

Je n'ai pas la notice @Jireck-npds tu as modifier grab_globals.php ?

Non ! Toutes les remarques que je fais en retour de test sont sans modif juste du parametrage standard

Jireck-npds commented 6 years ago

je ne suis pas spécialiste du file manager mais il me semble que ce comportement est normal

si il n'y a pas d'autorisation picmanager (le petit fichier) on ne voit que les répertoires

si il y a autorisation on voit les fichiers images ...

Il est dommage que le compte admin créé lors de l'install n'est pas d'autorisation un minimum et faire une petite notification avec les droits et un lien vers l'aide

jipexu commented 6 years ago

Oui moi aussi je n'avais pas la même ligne commentée pour le report d'erreurs grab_globals.php Néanmoins en me mettant la même chose que labo cela ne change rien (pour moi), j'ai toujours les mêmes notices.

Petit Rectificatif: j'ai avec la précédente notice (...on line 86) bien l'affichage des images .... (il m'a fallu scrooller un moment cette répétition de même notice pour les voir en bas de page)

est ce que tu peux reproduire l'erreur de la ligne 86 sur le labo ???

previnfo commented 6 years ago

Hi, Avec tes dernières modifs (25 & 26/10), que j'ai reporté chez moi, tout est fonctionnel... => Plus aucune notices, et les images s'affichent... !

jipexu commented 6 years ago

je ne vois pas le rapport avec l'install et le compte admin ...? pour l'instant il n'existe que deux dossiers dans l'archive qui embarque un pic-manager.txt celui de l'utilisateur par defaut users_private/user/pic-manager.txt et celui de la matrice des groupes modules/groupe/matrice/images_groupe/pic-manager.txt

oui on a pas d'aide : peut être rajouter un onglet avec lien sur l'aide ...

jipexu commented 6 years ago

@previnfo mais tu dois toujours avoir (et ce que sur le dossier avatar) Notice: Undefined index: gene-img in /home/users/i/infocapagde/www/NPDS/labo/modules/f-manager/pic-manager.php on line 325 ... ??

previnfo commented 6 years ago

Oui exact la cause => ma ligne commentée pour le report d'erreurs grab_globals.php était revenu par défaut.... (?) J'ai remis les mêmes que labo & oui la notice revient .

jipexu commented 6 years ago

c'est un truc tordu lol (en fait le fichier blank avatar est un fichier png alors qu'il a une extension gif lol ..) du coup quand la fonction essai de créer un thumbnail en croyant avoir affaire à un fichier gif elle plante logiquement et comme l'erreur est masqué par le @ on se retrouve avec un tableau incomplet ou il manque justement l'index gene_img ... donc je vais changer ce fichier et mettre un control sur la clef du tableau ... (et ce devrait suffire mais ne pas empêcher ce type de subterfuge ... ) et c'est bibi qui a probablement fait cette petite manipulation quand on a changer le fichier blank ...

jipexu commented 6 years ago

au passage j'ai modifié la fonctionCreateThumb() dans le picmanager.php pour qu'elle puisse créér aussi les thumbnails en png on a donc à peu près la même fonction que dans npds-galerie qu'on avait fait évolué pour le support png ...

Jireck-npds commented 6 years ago

je ne vois pas le rapport avec l'install et le compte admin ...? pour l'instant il n'existe que deux dossiers dans l'archive qui embarque un pic-manager.txt celui de l'utilisateur par defaut users_private/user/pic-manager.txt et celui de la matrice des groupes modules/groupe/matrice/images_groupe/pic-manager.txt

oui on a pas d'aide : peut être rajouter un onglet avec lien sur l'aide ...

Mon point de vue c'est que lors d'une install, on déclare un compte admin. Généralement ensuite on se connecte avec ce compte pour faire de la configuration, etc... Comme l'onglet est présent, on se demande, logiquement avec son nom, si on peux gere mieux ou différemment les images. et ensuite tu te balade dans l'arborescence ou tu ne vois justement que l'arborescence. Pour moi c'est pas normalement soit :

tu as pas l'onglet ou un lien vers configurer son pic-manager (il est admin il peut).
Tu as l'onglet et ton pic-manager est fait en meme temps que le premier compte admin.

Jireck-npds commented 6 years ago

au passage j'ai modifié la fonctionCreateThumb() dans le picmanager.php pour qu'elle puisse créér aussi les thumbnails en png on a donc à peu près la même fonction que dans npds-galerie qu'on avait fait évolué pour le support png ...

Pour info j'avais deja fait cette fonction pour TD-Gal et dans certains parties du main integré.

jipexu commented 6 years ago

Pour info j'avais deja fait cette fonction pour TD-Gal et dans certains parties du main integré.

oui c'est pour cette raison que j'ai écrit

.... qu'on avait fait évolué pour le support png ...

jipexu commented 6 years ago

tu as pas l'onglet ou un lien vers configurer son pic-manager (il est admin il peut). Tu as l'onglet et ton pic-manager est fait en meme temps que le premier compte admin.

comprend pas ...

Jireck-npds commented 6 years ago

Je reformule : A quoi sert d'avoir un onglet "image manager" si c'est la même chose que file manager ?

Donc prenons un peu de hauteur. Le Compte Admin de l'installation est par défaut le plus élevé privilège avant avec ce compte créer d'autres comptes admin avec un droit superadmin. il doit donc avoir toutes les fonctions actives ou activable facilement.

Avec cette information ci-dessus, pourquoi ne pas avoir les choses suivantes : 1, "image manager" activé en même temps que la déclaration du compte admin dans l'installation. 2, avoir l'onglet mais qui pour les admins, qui pointe soit vers la gestion / création de "image manager" soit c'est actif donc comme actuel.

jipexu commented 6 years ago

image manager et file manager c'est pas la même chose (actuellement)

dans file manager tu as toutes les fonctions relatives aux dossiers et fichiers (y compris celle d'autoriser le pic manager pour un dossier ...) dans image manager tu n'as que la navigation entre les répertoires et si ceux ci ont le pic manager activé alors tu vois toutes tes images et que les images ... donc y'a quand même une différence non ?

Donc prenons un peu de hauteur. Le Compte Admin de l'installation est par défaut le plus élevé privilège avant avec ce compte créer d'autres comptes admin avec un droit superadmin. il doit donc avoir toutes les fonctions actives ou activable facilement.

Avec cette information ci-dessus, pourquoi ne pas avoir les choses suivantes : 1, "image manager" activé en même temps que la déclaration du compte admin dans l'installation. 2, avoir l'onglet mais qui pour les admins, qui pointe soit vers la gestion / création de "image manager" soit c'est actif donc comme actuel.

... La tu as pris beaucoup trop de hauteur pour moi ...

Je ne vois toujours pas ce que vient faire le ou les admins la dedans ... tu veux que tous les dossier est une autorisation pour pic manager ????

Jireck-npds commented 5 years ago

Pour redescendre un admin ( et surtout le premier. seul compte admin) doit avoir tous les droits. Il a tous les droits sur File manager et en changeant d'onglet on a plus rien (puisque pas actif) Cela me semble pas logique. voila la logique pour les admins.

Je suggere que le pic manager soit au moins activé, pour le compte admin, sur le repertoire images et sous rep

stale[bot] commented 5 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

npds / npds_dune

Image manager #378