When saving a context and entering an invalid name, all changes to reactions and conditions are lost when the form fails validation. I don't know if there is a way to pass back the form / widget state if the save fails validation?

CommentFileSizeAuthor
#2 context-validation-952166-2.patch2.45 KBfebbraro

Comments

dealancer’s picture

dealancer commented

subscribing, this bug is also typical for 6.x version of this module.

febbraro’s picture

febbraro commented
Status: Active » Needs review
StatusFileSize
new context-validation-952166-2.patch2.45 KB

OK, this is a good first pass at repopulating the form on a validation error. There is a kink here or there (I think it does not preserve the debug reaction).

febbraro’s picture

febbraro commented
Status: Needs review » Closed (fixed)

Ok fixed and in git.

http://drupalcode.org/project/context.git/commit/1585880

caschbre’s picture

caschbre commented

Was that patched for the D7 version, D6 version, or both? The D6 dev version didn't look to be updated on the project page. I'm experiencing this issue with D6.

And it that is a D7 patch... would it also work in D6?

febbraro’s picture

febbraro commented
Version: 7.x-3.0-alpha2 » 6.x-3.0
Status: Closed (fixed) » Patch (to be ported)

This is fixed in D7, it could be back ported. I'm not sure how much things have changed b/w D6 & D7, but if you look at the diff from the git commit it could possibly be backported.

  • febbraro committed 1585880 on 8.x-4.x 
    #952166 Repopulate form on validation error

  • febbraro committed 1585880 on 8.x-0.x 
    #952166 Repopulate form on validation error

  • febbraro committed 1585880 on 8.x-1.x 
    #952166 Repopulate form on validation error
paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented
Issue summary: View changes

As this issue was already fixed and context for Drupal 6 is no longer supported, I'm closing this issue.
Thanks.

paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented
Status: Patch (to be ported) » Fixed
paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented
Status: Fixed » Closed (fixed)