#71 - editing default hardware definition allowed. Commented a block of code that I think it is not necessary. Need to be tested properly.

Author: stebjan

src/main/java/cz/zcu/kiv/eegdatabase/wui/ui/lists/form/HardwareFormPage.java

@@ -173,39 +173,40 @@ protected void onSubmit(AjaxRequestTarget target, Form<?> form) {
                         // Editing one
                         log.debug("Editing existing hardware object.");
 
-                        if (facade.isDefault(hardwareId)) {
-
-                            if (researchGroupId != CoreConstants.DEFAULT_ITEM_ID) {
-                                // new hardware
-                                Hardware newHw = new Hardware();
-                                newHw.setDefaultNumber(0);
-                                newHw.setDescription(hardware.getDescription());
-                                newHw.setTitle(hardware.getTitle());
-                                newHw.setType(hardware.getType());
-                                int newId = facade.create(newHw);
-                                HardwareGroupRel rel = facade.getGroupRel(hardwareId, researchGroupId);
-                                // delete old rel, create new one
-                                HardwareGroupRelId newRelId = new HardwareGroupRelId();
-                                HardwareGroupRel newRel = new HardwareGroupRel();
-                                newRelId.setHardwareId(newId);
-                                newRelId.setResearchGroupId(researchGroupId);
-                                newRel.setId(newRelId);
-                                newRel.setHardware(newHw);
-                                newRel.setResearchGroup(group);
-                                facade.deleteGroupRel(rel);
-                                facade.createGroupRel(newRel);
-                            } else {
-                                if (!facade.hasGroupRel(hardwareId) && facade.canDelete(hardwareId)) {
-                                    facade.update(hardware);
-                                } else {
-                                    getFeedback().error(ResourceUtils.getString("text.itemInUse"));
-                                    this.setEnabled(true);
-                                    return;
-                                }
-                            }
-                        } else {
-                            facade.update(hardware);
-                        }
+//                        if (facade.isDefault(hardwareId)) {
+//
+//                            if (researchGroupId != CoreConstants.DEFAULT_ITEM_ID) {
+//                                // new hardware
+//                                Hardware newHw = new Hardware();
+//                                newHw.setDefaultNumber(0);
+//                                newHw.setDescription(hardware.getDescription());
+//                                newHw.setTitle(hardware.getTitle());
+//                                newHw.setType(hardware.getType());
+//                                int newId = facade.create(newHw);
+//                                HardwareGroupRel rel = facade.getGroupRel(hardwareId, researchGroupId);
+//                                // delete old rel, create new one
+//                                HardwareGroupRelId newRelId = new HardwareGroupRelId();
+//                                HardwareGroupRel newRel = new HardwareGroupRel();
+//                                newRelId.setHardwareId(newId);
+//                                newRelId.setResearchGroupId(researchGroupId);
+//                                newRel.setId(newRelId);
+//                                newRel.setHardware(newHw);
+//                                newRel.setResearchGroup(group);
+//                                facade.deleteGroupRel(rel);
+//                                facade.createGroupRel(newRel);
+//                            } else {
+//                                if (!facade.hasGroupRel(hardwareId) && facade.canDelete(hardwareId)) {
+//                                    facade.update(hardware);
+//                                } else {
+//                                    getFeedback().error(ResourceUtils.getString("text.itemInUse"));
+//                                    this.setEnabled(true);
+//                                    return;
+//                                }
+//                            }
+//                        } else {
+//
+//                        }
+                        facade.update(hardware);
                     } else {
 
                         // Creating new