[RC] My Topics 1.0.9 [compatible 3.2.x]

Pour les extensions destinées à phpBB 3.1.x

Modérateur: Equipe

Règles du forum
A lire impérativement : Règlement de phpBB-fr.com

[RC] My Topics 1.0.9 [compatible 3.2.x]

Messagepar pierredu » 02 Oct 2015 à 14:16

Bonjour,

  • Titre : My Topics
  • Description : Addition d'une rubrique Mes sujets dans le menu Accès rapide
  • Version : 1.0.4
  • Auteur : Pierre Duhem
  • Historique des versions : Version 1.0.3 (résultat des remarques faites sur le sujet)

https://github.com/pierrdu/lmdi_mytopics

J'ai résolu tous mes problèmes ou presque.
Le code de chargement de la langue ne semble pas fonctionner, parce que j'obtiens une chaîne MY_TOPICS dans l'affichage, alors que ce devrait être Mes sujets.

Sinon, l'utilité d'une telle extension est de permettre de voir tous les sujets créés par l'utilisateur et non pas tous les messages, qui peuvent être dispersés entre de nombreux sujets.
Modifié en dernier par pierredu le 02 Avr 2016 à 17:10, modifié 4 fois.
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1051
Enregistré le: 29 Mai 2011 à 06:49
Localisation: Paris

Re: My Topics 1.0.0

Messagepar galixte » 03 Oct 2015 à 07:58

Salut,

intéressant. ;=)

C’est corrigé ton affichage de la clé de langue depuis que Richard t’a corrigé ton fichier ou je me trompe ?
Modifié en dernier par galixte le 31 Oct 2015 à 13:49, modifié 1 fois.
Communau EzCom
Image
« Traductions d’extensions & styles pour phpBB 3.1.x & 3.2.x ».
ImageListes de toutes les extensions recensées pour phpBB 3.1.x & 3.2.x.
ImageListes de tous les styles recensés pour phpBB 3.1.x & 3.2.x.

Image Tu as un forum et tu veux aussi un site web ? Regarde par ici.
Avatar de l’utilisateur
galixte
Floodeur universel
Floodeur universel
 
Messages: 10124
Enregistré le: 08 Mar 2004 à 02:40
Localisation: Sud de la France.

Re: My Topics 1.0.0

Messagepar pierredu » 03 Oct 2015 à 09:29

Oui, c'est corrigé.
Le code qu'il m'a donné pour charger les chaînes de langue est sensiblement différent de celui utilisé par Cabot dans son extension.
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1051
Enregistré le: 29 Mai 2011 à 06:49
Localisation: Paris

Re: My Topics 1.0.0

Messagepar cabot » 03 Oct 2015 à 12:10

Hello,


  • Il faudrait conditionner l'affichage du lien avec S_DISPLAY_SEARCH, comme c'est le cas pour le lien vers les messages, pour respecter les critères de permissions.
    Code: Tout sélectionner
    <!-- IF S_DISPLAY_SEARCH and S_REGISTERED_USER -->

  • Pourquoi ajouter le SID dans le lien ?

  • Tu déclares une balise i ayant pour classe icon-uniE0C6 mais cette classe n'est pas utilisée et la balise et vide, c'est un reste de c/c ou tu prévois de la déco ? :)
Image
Avatar de l’utilisateur
cabot
Equipe site
Equipe site
 
Messages: 26796
Enregistré le: 13 Fév 2011 à 20:42
Localisation: Rarement sur la plaque

Re: My Topics 1.0.0

Messagepar pierredu » 03 Oct 2015 à 15:16

Salut,

Bonnes questions.

Sur la double condition, tu as raison. Je vais corriger.

Sur l'inclusion du SID dans le lien : si tu ne le fais pas, tu sors de la session et il faut rouvrir une session. Autrement dit, la recherche se fait dans la session, mais l'affichage se fait en dehors de celle-ci. Cela pose notamment un problème quand tu es modérateur/animateur ou même administrateur (quelle que soit la terminologie réellement utilisée sur un forum donné, je crois qu'on fait ça à l'envers sur le nôtre).

Balise : j'ai simplement pompé la ligne que tu donnais dans l'un des sujets ci-dessous. Je vais corriger, ça ne sert à rien.
http://forums.phpbb-fr.com/support-utilisation-phpbb31/sujet206152.html
http://forums.phpbb-fr.com/message1607262.html
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1051
Enregistré le: 29 Mai 2011 à 06:49
Localisation: Paris

Re: My Topics 1.0.0

Messagepar Zoddo » 03 Oct 2015 à 15:18

cabot a écrit:Pourquoi ajouter le SID dans le lien ?

En faites, l'URL est mal construite. Tu ne dois pas construire les URLs dans les templates mais dans le code PHP et ensuite injecter l'URL dans une variable de template que tu peux utiliser.

Edit : Je répondais pendant que tu as posté ta réponse.
pierredu a écrit:Sur l'inclusion du SID dans le lien : si tu ne le fais pas, tu sors de la session et il faut rouvrir une session.
C'est justement le but de la fonction append_sid() : Ajouter le paramètre SID lorsque c'est nécessaire afin de ne pas perdre la session.
[Doc] Installer une extension - [Extension] Post Models - [Recrutement] Équipe des Extensions
Si vous considérez votre requête comme résolue, merci d'ajouter l'attribut de sujet [Réglé]
Zoddo
Equipe site
Equipe site
 
Messages: 6588
Enregistré le: 16 Mar 2011 à 18:44
Localisation: Laval / Le Mans

Re: My Topics 1.0.0

Messagepar pierredu » 03 Oct 2015 à 15:27

Exact.

Mais là, nous avons une extension minimale. Pas de listener, pas de code, juste une ligne incluse dans le gabarit d'affichage.

J'aurais pu coller ma ligne de code dans navbar_header.php et ni vu ni connu.

J'ai voulu essayer de jouer en respectant les règles du jeu.

La notion d'extensions est intéressante.
La seule limite est que l'on peut rajouter des éléments, mais pas en supprimer.
Je ne sais pas comment on pourrait le faire dans ce modèle de programmation. Retourner un code nul ou non nul à certains événements pour laisser inclure le bloc qui suit ou pas ?
J'ai le problème dans le gabarit posting_editor.html.
Sur mon forum, tout le monde se balance de savoir si le bbcode est autorisé ou non, etc. (voir au-dessous des smileys sur la droite de la zone d'édition).
Par contre, il est utile d'avoir à cet endroit un lien vers une page de téléchargement d'une image pour illustrer le propos de l'auteur du message.
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1051
Enregistré le: 29 Mai 2011 à 06:49
Localisation: Paris

Re: My Topics 1.0.0

Messagepar cabot » 03 Oct 2015 à 16:16

Vu pour la balise, effectivement elle ne servira pas car la modif proposée en lien concerne un style qui charge une librairie d'icônes vectorielles affichées au travers de ces balises et classes.
Image
Avatar de l’utilisateur
cabot
Equipe site
Equipe site
 
Messages: 26796
Enregistré le: 13 Fév 2011 à 20:42
Localisation: Rarement sur la plaque

Re: My Topics 1.0.0

Messagepar pierredu » 03 Oct 2015 à 17:01

Cabot,

Merci encore.
Je vais corriger le code.
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1051
Enregistré le: 29 Mai 2011 à 06:49
Localisation: Paris

Re: My Topics 1.0.0

Messagepar pierredu » 18 Oct 2015 à 19:04

Bonsoir,

Encore un truc que je viens de découvrir.
J'avais essayé de mettre ma ligne en haut du menu Accès rapide, mais ça ne marchait pas. Donc je l'ai mis en bas, ce qui marche.

En faisant des essais avec l'extension Pages, je me suis aperçu que j'avais le même problème (en haut ça ne marche pas, en bas ça marche). Après une question posée sur le forum phpbb.com, j'ai dû constater que ça ne marchait pas à cause de mon style. Si je reviens au style prosilver, cela peut se mettre à la première ligne.

Mon style avait été fait par copie de prosilver 3.1.3 et essentiellement modification des couleurs. J'ai comparé tous les fichiers avec Winmerge et je ne vois pas de réelle différence.

Pour quelqu'un utilisant prosilver, il serait donc possible de remplacer navbar_header_quick_links_after par navbar_header_quick_links_before.
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1051
Enregistré le: 29 Mai 2011 à 06:49
Localisation: Paris

Re: My Topics 1.0.0

Messagepar Zoddo » 18 Oct 2015 à 19:19

L'event navbar_header_quick_links_before est inclus dans votre style ?
[Doc] Installer une extension - [Extension] Post Models - [Recrutement] Équipe des Extensions
Si vous considérez votre requête comme résolue, merci d'ajouter l'attribut de sujet [Réglé]
Zoddo
Equipe site
Equipe site
 
Messages: 6588
Enregistré le: 16 Mar 2011 à 18:44
Localisation: Laval / Le Mans

Re: My Topics 1.0.0

Messagepar pierredu » 18 Oct 2015 à 21:48

Non, parce que ça ne marchait pas.
Mais j'indiquais cela à l'attention d'un utilisateur qui aurait comme style prosilver.
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1051
Enregistré le: 29 Mai 2011 à 06:49
Localisation: Paris

Re: My Topics 1.0.0

Messagepar Zoddo » 28 Oct 2015 à 16:04

Salut,

Comme dit plus haut, ton extension est invalide à cause de l'URL construite dans les templates. Elle ne pourra donc pas être validé en l'état dans une base de données d'extensions.

Tu dois construire ton URL dans le code PHP puis ensuite l'injecter dans une variable de template. En plus, ton URL ne fonctionne pas si tu es sur une page qui n'est pas à la racine du forum (exemple : essaie de cliquer sur ton lien depuis la page de reconnexion à l'ACP).
[Doc] Installer une extension - [Extension] Post Models - [Recrutement] Équipe des Extensions
Si vous considérez votre requête comme résolue, merci d'ajouter l'attribut de sujet [Réglé]
Zoddo
Equipe site
Equipe site
 
Messages: 6588
Enregistré le: 16 Mar 2011 à 18:44
Localisation: Laval / Le Mans

Re: My Topics 1.0.0

Messagepar pierredu » 28 Oct 2015 à 21:59

Merci de tes remarques.
Je vais en tenir compte.

Le lendemain matin :
Pour le problème de l'accès depuis l'accès au panneau d'administration, les autres rubriques, qui sont normalement du type ./, sont codées ./../, pour remonter d'un cran.
Reste à trouver comment faire.

Le lendemain midi :
Voilà, c'est fait pour l'injection de l'URL. Il me reste à trouver une solution pour connaître le niveau de l'arborescence auquel je me trouve pour corriger en conséquence l'url.
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1051
Enregistré le: 29 Mai 2011 à 06:49
Localisation: Paris

Re: My Topics 1.0.0

Messagepar pierredu » 30 Oct 2015 à 17:03

Bon, j'ai fini par trouver avec l'aide d'un développeur sur phpbb.com.
Il suffisait de composer l'url de la manière suivante :
Code: Tout sélectionner
$url = append_sid ($phpbb_root_path . "search." . $phpEx, $params);


Merci encore.
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1051
Enregistré le: 29 Mai 2011 à 06:49
Localisation: Paris

Suivante

Retourner vers Extensions en développement

 


  • Articles en relation
    Réponses
    Vues
    Dernier message

Qui est en ligne

Utilisateurs parcourant ce forum : Aucun utilisateur enregistré et 1 invité

cron