Changeset 1223

Show
Ignore:
Timestamp:
30-11-10 15:56:07 (3 years ago)
Author:
t.w.abma@…
Message:

- improved error messaging in the last step of the wizard

Location:
trunk/grails-app
Files:
3 modified

Legend:

Unmodified
Added
Removed
  • trunk/grails-app/controllers/dbnp/importer/ImporterController.groovy

    r1216 r1223  
    377377            render(view:"step4", model:[validatedSuccesfully:session.validatedSuccesfully, failedtopersist:failedtopersist, updatedentities:updatedEntities, totalrows:session.importer_importeddata.size, referer: session.import_referer]) 
    378378            session.import_wizard_init = false 
    379  
    380             println "failed:" 
    381             failedtopersist.each { 
    382                 println it 
    383             } 
    384379        } 
    385380    } 
  • trunk/grails-app/services/dbnp/importer/ImporterService.groovy

    r1207 r1223  
    283283        def entitystored = null 
    284284        def failedtopersist = [] 
     285        def persisterrors = [] 
    285286        def updatedentities = [] 
    286287        study.refresh() 
     
    292293                        case Study       :  print "Persisting Study `" + entity + "`: " 
    293294                                                entity.owner = AuthenticationService.getLoggedInUser() 
     295                                                println "storing study" + entity 
     296                                                println "dump=" + entity.dump() 
     297                                                println "validate=" + entity.validate() 
    294298                                                if (persistEntity(entity)) validatedSuccesfully++; 
    295299                                                    else failedtopersist.add(entity) 
     
    324328                                                    study.addToSamples(entity) 
    325329                                                    if (persistEntity(study)) validatedSuccesfully++;                                                        
    326                                                 } else failedtopersist.add(entity) 
     330                                                } else { 
     331                                                    failedtopersist.add(entity) 
     332                                                } 
    327333                                                 
    328334                                                break 
     
    396402     */ 
    397403    boolean persistEntity(entity) { 
    398             println "persisting ${entity}"             
     404            println "persisting ${entity}"         
    399405            // if not validated 
    400406                if (entity.validate()) { 
  • trunk/grails-app/views/importer/step4.gsp

    r1207 r1223  
    3131      <p>The following entities could not be persisted:</p> 
    3232      <table>      
    33       <g:each var="entity" in="${failedtopersist}"> 
     33      <g:each var="entity" in="${failedtopersist}">        
    3434        <tr> 
    3535        <g:each var="field" in="${entity.giveFields()}"> 
     
    4242          </td> 
    4343        </g:each> 
     44        <td> 
     45           <g:each var="error" in="${entity.errors.allErrors}"> 
     46             <b>error</b>: field `${error.getField()}` rejected value: ${error.getRejectedValue()}</b> 
     47        </g:each> 
     48        </td> 
    4449      </tr> 
    4550      </g:each>