Ticket #676 (closed bug: fixed)

Opened 2 years ago

Last modified 2 years ago

jForm : format d'input non/mal vérifié

Reported by: foxmask Owned by: Julien
Priority: normal Milestone: Jelix 1.1 beta 1
Component: jelix:forms Version: trunk
Severity: minor Keywords: jforms datetime htmleditor
Cc: Php version:
Review: Hosting Provider:
Blocked By: #686, #687 Documentation needed: no
Blocking:

Description

Bonjour, (version de Jelix utilisée : 1.1alpha)

Dans mon formulaire je veux que soit obligatoire la saisi de mon champ utilisant un editeur html seulement lors de la soumission du formulaire, après avoir saisi du texte dans le champ, j'obtiens une popup me demandant de saisir le champ. Si je resoumets sans rien faire de plus, le formulaire est "passé" .

voici le noeud correspond dans le fichier .form.xml

<htmleditor ref="news_content" required="true">
<label>Contenu de votre News</label>
<alert type="required">le texte de votre actualité ne peut être vide</alert>
</htmleditor>

Ensuite dans un champ ou la date peut être entrée, le format n'est pas vérifié, aucune popup ne s'affiche pour me dire que le format n'est pas correct.

voici le noeud correspond dans le fichier .form.xml

<input ref="news_date" type="date" required="true">
<label>Date de publication</label>
<alert type="required">la date est obligatoire</alert>
<alert type="invalid">le format de la date est invalide</alert>
</input>

cordialement.

Attachments

foxmask.zip (120.8 kB) - added by foxmask 2 years ago.
Minuscule application pour montrer le comportement des 2 champs non vérifiés

Change History

Changed 2 years ago by foxmask

Minuscule application pour montrer le comportement des 2 champs non vérifiés

Changed 2 years ago by laurentj

  • version set to trunk
  • milestone jelix 1.1 deleted

Changed 2 years ago by Julien

  • status changed from new to assigned
  • owner set to Julien

Hello,

je n'avais pas vu ce ticket avant...

le bug avec la date, je viens de le fixer avec le ticket #686

je n'arrive pas à reproduire le bug du wysiwyg, ici tout fonctionne (svn1077)

Changed 2 years ago by foxmask

bonjour, j'ai identifié le soucis avec FF 3.0.1. pas testé avec IE par contre.

si vous n'avez pas le soucis avec FF 3.0.1 à votre tour, vous pouvez fermé le ticket.

cordialement.

Changed 2 years ago by Julien

Ok,

ici FFOX 3.0.1 Linux

le problème vient à priori des scripts JS de jForms qui sont trop vieux, ou du moins pas à jour

remplaces le répertoire app/www/jelix par lib/jelix-www (si tu es sur linux, un lien symbolique permet de tjrs avoir les fichiers à jour et de ne pas dupliquer appli par appli les ressources www de jelix).

donc normalement c'est fixé, merci de me remonter l'info pour que je ferme le ticket si tout fonctionne de ton côté.

Changed 2 years ago by foxmask

Voici la suite de mes tests.

Le soucis du wysiwyg persiste apres avoir récupéré la nightly build de ce jour.

image pour illustrer le comportement quand le formulaire est entierement rempli.

Problème avec les browsers suivants :

FF 3.0.1 Anglais sous XP Pro / Extensions existantes : firebug/html validator/webdevelopper

Flock 1.2 (utilisant le moteur FF 2.x) sans aucune extension

Pas de Problème avec le browser suivant :

IE 6 (SP2)

cordialement.

Changed 2 years ago by Julien

Ok j'ai trouvé le problème. Le textarea correspondant au wymeditor n'est pas mis à jour à temps... En effet, il semble que le check du form intervienne avant que l'évènement submit soit transmis au wysiwyg, qui donc ne met à jour le textarea que la seconde fois.

Je vais essayer de fixer, ça semble pas trop compliqué. Patch cet aprem sans doute.

Changed 2 years ago by foxmask

merci !

mais rien ne presse ;-)

Changed 2 years ago by Julien

sisi car je dois sortir une appli avec du wysiwyg d'ici demain ;)

j'ai écris un patch qui fonctionne, mais uniquement pour wymeditor.

je vais faire un truc plus générique, afin que ça supporte les autres éditeurs comme tinymce, etc...

Changed 2 years ago by Julien

Ok, j'ai ouvert un ticket spécifique au bug wymeditor : #687

Changed 2 years ago by Julien

pour info, #687 dépend de #688 à présent.

il faut donc valider #688, puis #687 pour fermer le ticket

Changed 2 years ago by laurentj

  • milestone set to Jelix 1.1 beta 1

Changed 2 years ago by Julien

  • status changed from assigned to closed
  • resolution set to fixed

#687 is fixed, so this one is fixed too

Note: See TracTickets for help on using tickets.