Menu

#633 PropertySet validation does not work correctly anymore

0.9.4-beta
closed-fixed
JFireBase (268)
5
2008-07-21
2008-07-21
jfirechief
No

DO NOT EDIT OR ANSWER THIS ISSUE. SEE THE ORIGINAL ISSUE INSTEAD:
https://www.jfire.org/modules/bugs/view.php?id=692
ORIGINAL REPORTER: Marc

Opening the user francois there should be initial errors (everything is empty). This already worked.

An empty email address field should be valid and saveable.

The internet (email) validation should not popup a dialog but show a message at the top. And by the way, the dialog that pops up is not a JFace MessageDialog!

Discussion

  • jfirechief

    jfirechief - 2008-07-21
    • labels: --> JFireBase
    • milestone: --> 0.9.4-beta
    • assigned_to: nobody --> fleque
    • status: open --> closed-fixed
     
  • jfirechief

    jfirechief - 2008-07-21

    Logged In: YES
    user_id=1828570
    Originator: YES

    ORIGINAL COMMENT BY alex, VIEW IT HERE:
    https://www.jfire.org/modules/bugs/view.php?id=692

    Validation is now also done at startup again.

     
  • jfirechief

    jfirechief - 2008-07-21

    Logged In: YES
    user_id=1828570
    Originator: YES

    ORIGINAL COMMENT BY alex, VIEW IT HERE:
    https://www.jfire.org/modules/bugs/view.php?id=692

    Concerning the validation of the email: It was not done through the validation framework of PropertySets. I tried to implement it there but it was not possible as the evaluation context does not give access to the StructField where the regex is stored.

    However, we decided to re-write the validation of porperty-sets making use of the scripting/expression frame work of JFireScripting somehow. I will create new issue for that.

    I'll simply disable the regex check for the release now.

     
  • jfirechief

    jfirechief - 2008-07-21

    Logged In: YES
    user_id=1828570
    Originator: YES

    ORIGINAL COMMENT BY alex, VIEW IT HERE:
    https://www.jfire.org/modules/bugs/view.php?id=692

    Fixed what could be done quickly.
    Disabled validation of RegExDataFields.

     

Log in to post a comment.