Bonjour, je m'appelle Benjamin Gandon.

J'ai découvert Wikini grâce à free et voulant l'utiliser pour mon projet Wikipendium, j'ai été amené à apporter plusieurs modifications :

• support des tableaux,
  • Quel format de mise en forme utilises-tu? C'est le sujet de discussion quand on aborde le sujet des tableaux --GarfieldFr
• ajout de la coloration assembleur 68k,
• amélioration du comportement de %% avec une hauteur du cadre adaptée au contenu,
• le tunning des mots wikis pour s'adapter au contexte spécifique du projet.

Je suis d'accord pour prendre en main l'ajout du support des tableaux à Wikini. Pour ce faire, je pense me diriger vers un minimum de modifications pour un résultat simple et robuste. Je ne propose pas de refondre l'ensemble du code (c.f. OlivierB) pour y arriver.


Voici quelques commentaires qui me sont venus à la relecture du code de Wikini :

• beaucoup de regexp de Wikini sont améliorables :
  • Ecrire [A-Z,a-z,0-9] au lieu de [A-Za-z0-9] fait accepter à Wikini les Mots,Wiki contenants des virgules, ce qui est anormal.
    • [Corrigé dans la version 0.5-dev de Wikini -- DavidDelon]
  • La regexp des images /.(gif|jpeg|png|jpg)$/ devrait contenir un caractère anti-slash au début : /\.(gif|jpeg|png|jpg)$/ sans quoi l'usage du point est inutile.
  • Les adresses email sont reconnues à l'aie de /^.+\@.+$/ ce qui fait dire à Wikini que http://user@www.example.com est un lien de courriel, ce qui est anormal.
    Utiliser /^[\w.-]+\@[\w.-]+$/ serait plus précis et corrigerait ce bug.
    A ce sujet, j'ai justement modifié la page des CritiquesDesWikis pour contourner ce bug. Celui-ci y sévissait car le lien vers l'article du Monde contient un caractère @ et il ne fonctionne donc pas tel quel (j'ai corrigé cela en changeant @ par son équivalent %40).
  • Il est possible d'écrire /:\/\// plus lisiblement : |://|
    • En fait, il serait préférable de changer tous les / de début et fin de regexp par autre chose ( moi j'utilise ` car rarement utilisé )--GarfieldFr
  • J'observe des protections "avec ceinture et bretelles". Par exemple, dans la regexp pour les liens inter-wiki /^([A-Z][A-Z,a-z]+)[:]([A-Z,a-z,0-9]*)$/s qui (outre le bug des virgules) peut s'écrire plus simplement : /^([A-Z][A-Za-z]+):([A-Za-z0-9]*)$/s car le caractère "deux points" n'est pas un caractère spécial pour les regexp.

[Merci pour toutes ces remarques, nous allons essayer de les intégrer rapidement; l'équipe de developpement est assez ouverte, n'hésite pas à postuler si tu désires un accès CVS. -- DavidDelon]

---