Changeset 1149 for trunk


Ignore:
Timestamp:
Nov 16, 2010, 6:30:11 PM (11 years ago)
Author:
work@…
Message:
  • resolving issue #197 where the study wizard breaks is flash.errors does not contain a map which it expects. Probably because elsewhere in the application the flash.errors is also being set
Location:
trunk/grails-app
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/grails-app/controllers/dbnp/studycapturing/WizardController.groovy

    r1099 r1149  
    205205
    206206                                // reset errors
    207                                 flash.errors = [:]
     207                                flash.wizardErrors = [:]
    208208                                success()
    209209                        }.to "study"
    … …  
    213213
    214214                                // reset errors
    215                                 flash.errors = [:]
     215                                flash.wizardErrors = [:]
    216216                                success()
    217217                        }.to "study"
    … …  
    221221
    222222                                // reset errors
    223                                 flash.errors = [:]
     223                                flash.wizardErrors = [:]
    224224                                success()
    225225                        }.to "start"
    … …  
    264264
    265265                                // reset errors
    266                                 flash.errors = [:]
     266                                flash.wizardErrors = [:]
    267267
    268268                                // remove subject
    … …  
    277277
    278278                                // reset errors
    279                                 flash.errors = [:]
     279                                flash.wizardErrors = [:]
    280280                                success()
    281281                        }.to "study"
    … …  
    334334
    335335                                // reset errors
    336                                 flash.errors = [:]
     336                                flash.wizardErrors = [:]
    337337                                success()
    338338
    … …  
    351351
    352352                                // reset errors
    353                                 flash.errors = [:]
     353                                flash.wizardErrors = [:]
    354354                                success()
    355355                        }.to "events"
    … …  
    359359
    360360                                // reset errors
    361                                 flash.errors = [:]
     361                                flash.wizardErrors = [:]
    362362
    363363                                // add event to study
    … …  
    384384
    385385                                        // append errors
    386                                         this.appendErrors(flow.event, flash.errors)
     386                                        this.appendErrors(flow.event, flash.wizardErrors)
    387387                                        error()
    388388                                }
    … …  
    393393
    394394                                // reset errors
    395                                 flash.errors = [:]
     395                                flash.wizardErrors = [:]
    396396
    397397                                // find matching (sampling) event
    … …  
    429429
    430430                                // reset errors
    431                                 flash.errors = [:]
     431                                flash.wizardErrors = [:]
    432432                                success()
    433433                        }.to "events"
    … …  
    437437
    438438                                // reset errors
    439                                 flash.errors = [:]
     439                                flash.wizardErrors = [:]
    440440
    441441                                // remove eventGroup
    … …  
    450450
    451451                                // reset errors
    452                                 flash.errors = [:]
     452                                flash.wizardErrors = [:]
    453453                                success()
    454454                        }.to "subjects"
    … …  
    555555
    556556                                // ignore errors
    557                                 flash.errors = [:]
     557                                flash.wizardErrors = [:]
    558558
    559559                                succes()
    … …  
    569569
    570570                                // ignore errors
    571                                 flash.errors = [:]
     571                                flash.wizardErrors = [:]
    572572
    573573                                success()
    … …  
    583583
    584584                                // ignore errors
    585                                 flash.errors = [:]
     585                                flash.wizardErrors = [:]
    586586
    587587                                success()
    … …  
    592592
    593593                                // ignore errors
    594                                 flash.errors = [:]
     594                                flash.wizardErrors = [:]
    595595
    596596                                success()
    … …  
    622622
    623623                                // reset errors
    624                                 flash.errors = [:]
     624                                flash.wizardErrors = [:]
    625625                                success()
    626626                        }.to "assays"
    … …  
    641641
    642642                                // reset errors
    643                                 flash.errors = [:]
     643                                flash.wizardErrors = [:]
    644644                                success()
    645645                        }.to "assays"
    … …  
    649649
    650650                                // reset errors
    651                                 flash.errors = [:]
     651                                flash.wizardErrors = [:]
    652652
    653653                                // add assay to study
    … …  
    665665
    666666                                        // append errors
    667                                         this.appendErrors(flow.assay, flash.errors)
     667                                        this.appendErrors(flow.assay, flash.wizardErrors)
    668668                                        error()
    669669                                }
    … …  
    676676
    677677                                // reset errors
    678                                 flash.errors = [:]
     678                                flash.wizardErrors = [:]
    679679
    680680                                // find this assay
    … …  
    689689
    690690                                // ignore errors
    691                                 flash.errors = [:]
     691                                flash.wizardErrors = [:]
    692692
    693693                                success()
    … …  
    731731
    732732                                // ignore errors
    733                                 flash.errors = [:]
     733                                flash.wizardErrors = [:]
    734734
    735735                                success()
    … …  
    792792                        action {
    793793                                println "saving..."
    794                                 flash.errors = [:]
     794                                flash.wizardErrors = [:]
    795795
    796796                                // persist data to the database
    … …  
    803803
    804804                                        if (!flow.study.save(flush:true)) {
    805                                                 this.appendErrors(flow.study, flash.errors)
     805                                                this.appendErrors(flow.study, flash.wizardErrors)
    806806                                                throw new Exception('error saving study')
    807807                                        }
    … …  
    811811                                } catch (Exception e) {
    812812                                        // rollback
    813                                         this.appendErrorMap(['exception': e.toString() + ', see log for stacktrace' ], flash.errors)
     813                                        this.appendErrorMap(['exception': e.toString() + ', see log for stacktrace' ], flash.wizardErrors)
    814814
    815815                                        // stacktrace in flash scope
    … …  
    857857//              def authenticationService
    858858
    859                 flash.errors = new LinkedHashMap()
     859                flash.wizardErrors = new LinkedHashMap()
    860860               
    861861                // load study
    … …  
    880880                } catch (Exception e) {
    881881                        // rollback
    882                         this.appendErrorMap(['exception': e.getMessage() + ', see log for stacktrace'], flash.errors)
     882                        this.appendErrorMap(['exception': e.getMessage() + ', see log for stacktrace'], flash.wizardErrors)
    883883
    884884                        return false
    … …  
    945945                } else {
    946946                        // validation failed
    947                         flash.errors = [:]
    948                         this.appendErrors(flow.study, flash.errors)
     947                        flash.wizardErrors = [:]
     948                        this.appendErrors(flow.study, flash.wizardErrors)
    949949                        return false
    950950                }
    … …  
    11221122        def subjectPage(flow, flash, params) {
    11231123                def errors = false
    1124                 flash.errors = [:]
     1124                flash.wizardErrors = [:]
    11251125
    11261126                // remember the params in the flash scope
    … …  
    11411141                        if (!subject.validate()) {
    11421142                                errors = true
    1143                                 this.appendErrors(subject, flash.errors, 'subject_' + subject.getIdentifier() + '_')
     1143                                this.appendErrors(subject, flash.wizardErrors, 'subject_' + subject.getIdentifier() + '_')
    11441144                        }
    11451145                }
    … …  
    11691169
    11701170                // (re)set error message
    1171                 flash.errors = [:]
     1171                flash.wizardErrors = [:]
    11721172
    11731173                // set work variables
    … …  
    12131213
    12141214                                        // append errors
    1215                                         this.appendErrors(subject, flash.errors)
     1215                                        this.appendErrors(subject, flash.wizardErrors)
    12161216                                        errors = true
    12171217                                }
    … …  
    12201220                        // add feedback
    12211221                        errors = true
    1222                         if (number < 1) this.appendErrorMap(['addNumber': 'Enter a positive number of subjects to add'], flash.errors)
    1223                         if (!species)   this.appendErrorMap(['species': 'You need to select a species, or add one if it is not yet present'], flash.errors)
    1224                         if (!template)  this.appendErrorMap(['template': 'You need to select a template, or add one if it is not yet present'], flash.errors)
     1222                        if (number < 1) this.appendErrorMap(['addNumber': 'Enter a positive number of subjects to add'], flash.wizardErrors)
     1223                        if (!species)   this.appendErrorMap(['species': 'You need to select a species, or add one if it is not yet present'], flash.wizardErrors)
     1224                        if (!template)  this.appendErrorMap(['template': 'You need to select a template, or add one if it is not yet present'], flash.wizardErrors)
    12251225                }
    12261226
    … …  
    12381238        def eventPage(flow, flash, params) {
    12391239                def errors = false
    1240                 flash.errors = [:]
     1240                flash.wizardErrors = [:]
    12411241
    12421242                // remember the params in the flash scope
    … …  
    13501350                        if (!event.validate()) {
    13511351                                errors = true
    1352                                 this.appendErrors(event, flash.errors)
     1352                                this.appendErrors(event, flash.wizardErrors)
    13531353                        }
    13541354                }
    … …  
    13671367        def groupPage(flow, flash, params) {
    13681368                def errors = false
    1369                 flash.errors = [:]
     1369                flash.wizardErrors = [:]
    13701370
    13711371                // remember the params in the flash scope
    … …  
    14371437        def samplePage(flow, flash, params) {
    14381438                def errors = false
    1439                 flash.errors = [:]
     1439                flash.wizardErrors = [:]
    14401440
    14411441                // remember the params in the flash scope
    … …  
    14641464                        if (!sample.validate()) {
    14651465                                errors = true
    1466                                 this.appendErrors(sample, flash.errors, 'sample_' + sample.getIdentifier() + '_' )
     1466                                this.appendErrors(sample, flash.wizardErrors, 'sample_' + sample.getIdentifier() + '_' )
    14671467                                println 'error-> sample_'+sample.getIdentifier()
    14681468                        }
    … …  
    14821482        def assayPage(flow, flash, params) {
    14831483                def errors = false
    1484                 flash.errors = [:]
     1484                flash.wizardErrors = [:]
    14851485
    14861486                // remember the params in the flash scope
    … …  
    15111511                        if (!assay.validate()) {
    15121512                                errors = true
    1513                                 this.appendErrors(assay, flash.errors, 'assay_' + assay.getIdentifier() + '_')
     1513                                this.appendErrors(assay, flash.wizardErrors, 'assay_' + assay.getIdentifier() + '_')
    15141514                        }
    15151515                }
    … …  
    15281528        def assayGroupPage(flow, flash, params) {
    15291529                def errors = false
    1530                 flash.errors = [:]
     1530                flash.wizardErrors = [:]
    15311531
    15321532                // remember the params in the flash scope

  • trunk/grails-app/views/wizard/common/_error.gsp

    r959 r1149  
    1515         */
    1616%>
    17 <g:if test="${errors}">
     17<g:if test="${wizardErrors}">
    1818        <div id="wizardError" class="error" title="errors">
    19                 <g:each in="${errors}" var="error" status="e">
     19                <g:each in="${wizardErrors}" var="error" status="e">
    2020                        <p>
    2121                                <g:if test="${!e}"><span class="ui-icon ui-icon-alert" style="float:left; margin:0 7px 50px 0;"></span></g:if>
    … …  
    2626        <script type="text/javascript">
    2727                // mark error fields
    28                 <g:each in="${errors}" var="error">
     28                <g:each in="${wizardErrors}" var="error">
    2929                var element = $("input:[name='${error.key}'], input:[name='${error.key.toLowerCase().replaceAll("([^a-z0-9])","_")}'], select:[name='${error.key}'], select:[name='${error.key.toLowerCase().replaceAll("([^a-z0-9])","_")}']");
    3030                 <g:if test="${error.value['dynamic']}">
Note: See TracChangeset for help on using the changeset viewer.