Changeset 1492

Show
Ignore:
Timestamp:
04-02-11 11:16:53 (3 years ago)
Author:
t.w.abma@…
Message:

- corrected StringList? items in the mappings page are now also correctly removed (just like Ontology terms) from the failedcells flow variable

Location:
trunk/grails-app
Files:
4 modified

Legend:

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

    r1489 r1492  
    466466                                        } 
    467467                                } else { 
    468                                         removeFailedCell(flow.importer_failedcells, entity) 
     468                                        //removeFailedCell(flow.importer_failedcells, entity) 
    469469                                } // end else if 
    470470 
     
    472472                } // end of table 
    473473 
    474         println "invalidentities="+flow.importer_invalidentities 
    475  
    476474                return (flow.importer_invalidentities == 0) ? true : false 
    477475        } // end of method 
     
    483481        def removeFailedCell(failedcells, entity, field) { 
    484482                // Valid entity, remove it from failedcells 
     483        def entityidfield = "entity_" + entity.getIdentifier() + "_" + field.name.toLowerCase() 
     484 
    485485                failedcells.each { record -> 
    486486                        def tempimportcells = [] 
    487487 
    488                         record.importcells.each { cell -> 
    489                // println "cell_entity=" + cell.value 
    490                // println "fieldname=" + field.name 
    491  
     488 
     489                        record.importcells.each { cell ->                
    492490                                // remove the cell from the failed cells session 
    493                                 if (cell.entityidentifier != entity.getIdentifier()) { 
     491                                if (cell.entityidentifier != entityidfield) { 
    494492                                        //record.removeFromImportcells(cell) 
    495493                                        tempimportcells.add(cell) 
  • trunk/grails-app/domain/dbnp/importer/ImportCell.groovy

    r1457 r1492  
    1919class ImportCell extends Identity { 
    2020    MappingColumn mappingcolumn 
    21     int entityidentifier 
     21    String entityidentifier 
    2222    String value 
    2323 
  • trunk/grails-app/services/dbnp/importer/ImporterService.groovy

    r1489 r1492  
    299299 
    300300                                                study.addToSubjects(entity) 
     301                                        println "subject persisting" + entity.dump() 
    301302 
    302303                                                break 
     
    466467                                        log.error ".import wizard error could not set property `" + mc.property + "` to value `" + value + "`" 
    467468                                        // store the mapping column and value which failed 
    468                                         def identifier 
     469                    def identifier 
    469470 
    470471                                        switch (mc.entity) { 
    471                                                 case Study: identifier = study.getIdentifier() 
    472                                                         break 
    473                                                 case Subject: identifier = subject.getIdentifier() 
    474                                                         break 
    475                                                 case SamplingEvent: identifier = samplingEvent.getIdentifier() 
    476                                                         break 
    477                                                 case Event: identifier = event.getIdentifier() 
    478                                                         break 
    479                                                 case Sample: identifier = sample.getIdentifier() 
     472                                                case Study: identifier = "entity_" + study.getIdentifier() + "_" + mc.property 
     473                                                        break 
     474                                                case Subject: identifier = "entity_" + subject.getIdentifier() + "_" + mc.property 
     475                                                        break 
     476                                                case SamplingEvent: identifier = "entity_" + samplingEvent.getIdentifier() + "_" + mc.property 
     477                                                        break 
     478                                                case Event: identifier = "entity_" + event.getIdentifier() + "_" + mc.property 
     479                                                        break 
     480                                                case Sample: identifier = "entity_" + sample.getIdentifier() + "_" + mc.property 
    480481                                                        break 
    481482                                                case Object:   // don't import 
  • trunk/grails-app/views/importer/common/_missingproperties.gsp

    r1489 r1492  
    1818        <g:each in="${importer_failedcells}" var="record"> 
    1919          <g:each in="${record.importcells}" var="cell"> 
    20           var element = $("select[name=entity_${cell.entityidentifier}_${cell.mappingcolumn.property.toLowerCase()}]"); 
     20          var element = $("select[name=${cell.entityidentifier}]"); 
    2121 
    2222          element.addClass('error')