developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 12 years ago

Closed 12 years ago

#676 closed bug (fixed)

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: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

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 (1)

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

Download all attachments as: .zip

Change History (13)

Changed 12 years ago by foxmask

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

comment:1 Changed 12 years ago by laurentj

  • Milestone jelix 1.1 deleted
  • Version set to trunk

comment:2 Changed 12 years ago by Julien

  • Owner set to Julien
  • Status changed from new to assigned

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)

comment:3 Changed 12 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.

comment:4 Changed 12 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é.

comment:5 Changed 12 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.

comment:6 Changed 12 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.

comment:7 Changed 12 years ago by foxmask

merci !

mais rien ne presse ;-)

comment:8 Changed 12 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...

comment:9 Changed 12 years ago by Julien

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

comment:10 Changed 12 years ago by Julien

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

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

comment:11 Changed 12 years ago by laurentj

  • Milestone set to Jelix 1.1 beta 1

comment:12 Changed 12 years ago by Julien

  • Resolution set to fixed
  • Status changed from assigned to closed

#687 is fixed, so this one is fixed too

Note: See TracTickets for help on using tickets.