Changeset 719 for trunk/grails-app/services

Timestamp:

Nov 25, 2010, 8:47:00 AM (14 years ago)

Author:

gav

Message:

Domain change: as per ticket #96 - Remove unused fields from InventoryItem?.
Removed InventoryItem?.preferredManufacturer and manufacturersPartNumber.

File:

• trunk/grails-app/services/InventoryCsvService.groovy (modified) (11 diffs)

trunk/grails-app/services/InventoryCsvService.groovy

@@ -52 +52 @@
             def line = []
             def lineNumber = 0
-            def maxNumberOfColumns = 23
+            def maxNumberOfColumns = 20
             def inventoryParams = [:]
             def inventoryProperties = ["name", "description", "comment", "unitsInStock", "reorderPoint", "reorderQuantity",
@@ -59 +59 @@
                                                         "inventoryGroup", "inventoryType",
                                                         "suppliersPartNumber", "preferredSupplier", "alternateSuppliers",
-                                                        "manufacturersPartNumber", "preferredManufacturer", "alternateManufacturers",
                                                         "alternateItems", "spareFor"]
 
@@ -69 +68 @@
             def spareForInstance
             def alternateItemInstance
-            def preferredManufacturerInstance
-            def alternateManufacturerInstance
-            def manufacturerTypeInstance
-            def manufacturerTypeUnknown = ManufacturerType.get(1)
             def inventoryTypeInstance
             def unitOfMeasureInstance
@@ -88 +83 @@
             def tempSupplierType = ''
             def tempSupplierItemAndType = []
-
-            def tempPreferredManufacturerItemAndType = ''
-            def tempPreferredManufacturerItem = ''
-            def tempPreferredManufacturerType = ''
-
-            def tempAlternateManufacturers = []
-            def tempManufacturerItem = ''
-            def tempManufacturerType = ''
-            def tempManufacturerItemAndType = []
 
             def tempSpareFor = []
@@ -238 +224 @@
                         return fail(code: "inventory.import.failure", args: [lineNumber, logFileLink])
                     }
-                }
-
-                // Preferred Manufacturer
-                if(inventoryParams.preferredManufacturer) {
-                    tempPreferredManufacturerItemAndType = parseItemAndType(inventoryParams.preferredManufacturer)
-                    tempPreferredManufacturerItem = WordUtils.capitalize(tempPreferredManufacturerItemAndType[0])
-
-                    preferredManufacturerInstance = Manufacturer.findByName(tempPreferredManufacturerItem)
-                    if(!preferredManufacturerInstance) {
-
-                        // Manufacturer Type.
-                        if(tempPreferredManufacturerItemAndType.size == 2) {
-                            tempPreferredManufacturerType = WordUtils.capitalize(tempPreferredManufacturerItemAndType[1])
-                            manufacturerTypeInstance = ManufacturerType.findByName(tempPreferredManufacturerType)
-                        }
-                        else
-                            manufacturerTypeInstance = manufacturerTypeUnknown
-                        if(!manufacturerTypeInstance) {
-                            log.error "Failed to find preferred manufacturer type on line: " + lineNumber
-                            return fail(code: "inventory.import.failure", args: [lineNumber, logFileLink])
-                        }
-
-                        preferredManufacturerInstance = new Manufacturer(name: tempPreferredManufacturerItem,
-                                                                                                            manufacturerType: manufacturerTypeInstance)
-                        if(!preferredManufacturerInstance.save()) {
-                            log.error "Failed to create preferred manufacturer on line: " + lineNumber
-                            return fail(code: "inventory.import.failure", args: [lineNumber, logFileLink])
-                        }
-                    }
-                }
-                else
-                    preferredManufacturerInstance = null
-
-                // Alternate Manufacturers.
-                tempAlternateManufacturers = parseInputList(inventoryParams.alternateManufacturers)
-                inventoryParams.alternateManufacturers = []
-
-                for(tempManufacturer in tempAlternateManufacturers) {
-                    tempManufacturerItemAndType = parseItemAndType(tempManufacturer)
-                    tempManufacturerItem = WordUtils.capitalizeFully(tempManufacturerItemAndType[0])
-
-                    alternateManufacturerInstance = Manufacturer.findByName(tempManufacturerItem)
-                    if(!alternateManufacturerInstance) {
-
-                        // ManufacturerType.
-                        if(tempManufacturerItemAndType.size == 2) {
-                            tempManufacturerType = WordUtils.capitalize(tempManufacturerItemAndType[1])
-                            manufacturerTypeInstance = ManufacturerType.findByName(tempManufacturerType)
-                        }
-                        else
-                            manufacturerTypeInstance = manufacturerTypeUnknown
-                        if(!manufacturerTypeInstance) {
-                            log.error "Failed to find manufacturer type on line: " + lineNumber
-                            return fail(code: "inventory.import.failure", args: [lineNumber, logFileLink])
-                        }
-
-                        alternateManufacturerInstance = new Manufacturer(name: tempManufacturerItem,
-                                                                                                manufacturerType: manufacturerTypeInstance)
-                        if(!alternateManufacturerInstance.save()) {
-                            log.error "Failed to create manufacturers on line: " + lineNumber
-                            return fail(code: "inventory.import.failure", args: [lineNumber, logFileLink])
-                        }
-                    }
-
-                    inventoryParams.alternateManufacturers.add(alternateManufacturerInstance)
                 }
 
@@ -417 +338 @@
                 inventoryParams.unitOfMeasure = unitOfMeasureInstance
                 inventoryParams.preferredSupplier = preferredSupplierInstance
-                inventoryParams.preferredManufacturer = preferredManufacturerInstance
 
                 // Name.
@@ -485 +405 @@
         writer.writeNext("Comment: Required columns are marked with a (*) in the header line.")
         writer.writeNext("Comment: Lists of items in a column must be separated by a semicolon (;), not a comma.")
-        writer.writeNext("Comment: The at symbol (@) is reserved for indicating supplier and manufacturer types.")
+        writer.writeNext("Comment: The at symbol (@) is reserved for indicating supplier types.")
         writer.writeNext("Comment: Identical and existing names will be considered as the same item.")
         writer.writeNext("Comment: Lines containing 'comment' will be ignored.")
@@ -515 +435 @@
                                         "123", "Multi Supplier@Local",
                                         "Multi Distributors1@OEM; Multi Distributors2@Local",
-                                        "321", "Master Manufacturer@OEM",
-                                        "Mega Manufacturer1@OEM;Mega Manufacturer2@Alternate",
                                         "2204E-2RS", ""
                                         ] as String[])
@@ -530 +448 @@
                                         "456KL", "Multi Supplier",
                                         "Multi Distributors1; Multi Distributors2",
-                                        "654OP", "Master Manufacturer",
-                                        "Mega Manufacturer1;Mega Manufacturer2",
                                         "", ""
                                         ] as String[])
@@ -582 +498 @@
             }.collect { it.name + "@" + it.supplierType }.join(';') )
 
-            row.add(inventoryItem.manufacturersPartNumber)
-
-            if(inventoryItem.preferredManufacturer)
-                row.add( inventoryItem.preferredManufacturer.name + "@" + inventoryItem.preferredManufacturer.manufacturerType )
-            else
-                row.add('')
-
-            row.add(inventoryItem.alternateManufacturers.sort { p1, p2 ->
-                p1.name.compareToIgnoreCase(p2.name)
-            }.collect { it.name + "@" + it.manufacturerType }.join(';'))
-
-            row.add(inventoryItem.alternateItems.sort { p1, p2 ->
-                p1.name.compareToIgnoreCase(p2.name)
-            }.collect { it.name }.join(';') )
-
             row.add(inventoryItem.spareFor.sort { p1, p2 ->
                 p1.name.compareToIgnoreCase(p2.name)
@@ -813 +714 @@
             "Estimated Unit Price", "Currency", "Enable Reorder", "Location*", "Store*", "Site*", "Group*", "Type*",
             "Supplier's Part Number", "Preferred Supplier", "Alternate Suppliers",
-            "Manufacturer's Part Number", "Preferred Manufacturer", "Alternate Manufacturers", "Alternate Item", "Spare For"]
+            "Alternate Item", "Spare For"]
     }