#37: Implement changing IDs

kamil-sita · Jul 19, 2024 · d97049f · d97049f
1 parent 1e06b33
commit d97049f
Show file tree

Hide file tree

Showing 3 changed files with 49 additions and 3 deletions.
diff --git a/core/src/main/java/place/sita/labelle/core/repository/inrepository/InRepositoryService.java b/core/src/main/java/place/sita/labelle/core/repository/inrepository/InRepositoryService.java
@@ -21,7 +21,7 @@
 import place.sita.labelle.jooq.Tables;
 
 import javax.annotation.Nullable;
-import java.io.*;
+import java.io.File;
 import java.util.*;
 
 import static place.sita.labelle.jooq.Tables.*;
@@ -122,6 +122,35 @@ public UUID referImage(UUID newRepoId, UUID originalImageId, String persistentId
         return copyOrRefer(newRepoId, originalImageId, CopyOrRefer.REFER, persistentId);
     }
 
+    @Transactional
+    public UpdateIdsResult updateIds(UUID imageId, String persistentId, String parentPersistentId, boolean isVisibleToChildren) {
+        UUID imageRepoId = JqRepo.fetchOne(() ->
+            dslContext
+                .select(IMAGE.REPOSITORY_ID)
+                .from(IMAGE)
+                .where(IMAGE.ID.eq(imageId))
+                .fetch()
+        );
+        boolean exists = dslContext.fetchExists(IMAGE,
+            IMAGE.REFERENCE_ID.eq(persistentId)
+            .and(IMAGE.REPOSITORY_ID.eq(imageRepoId))
+            .and(IMAGE.ID.ne(imageId))
+        );
+        if (exists) {
+            return new UpdateIdsResult.IdReuse();
+        }
+
+        setPersistentId(imageId, persistentId);
+        setVisibility(imageId, isVisibleToChildren);
+        setParentPersistentId(imageId, parentPersistentId);
+        return new UpdateIdsResult.Success();
+    }
+
+    public sealed interface UpdateIdsResult {
+        record Success() implements UpdateIdsResult {}
+        record IdReuse() implements UpdateIdsResult {}
+    }
+
     @Transactional // todo make this one throw if something goes wrong - unless it already does?
     public void setPersistentId(UUID imageId, String persistentId) {
         dslContext.update(IMAGE)

diff --git a/...local/src/main/java/place/sita/labelle/gui/local/repositoryfx/PersistentIdController.java b/...local/src/main/java/place/sita/labelle/gui/local/repositoryfx/PersistentIdController.java
@@ -9,6 +9,7 @@
 import org.springframework.stereotype.Component;
 import place.sita.labelle.core.repository.inrepository.Ids;
 import place.sita.labelle.core.repository.inrepository.InRepositoryService;
+import place.sita.labelle.gui.local.fx.Alerts;
 import place.sita.modulefx.annotations.FxDictatesHeight;
 import place.sita.modulefx.annotations.FxMessageListener;
 import place.sita.modulefx.annotations.FxNode;
@@ -46,13 +47,27 @@ public class PersistentIdController {
     @ModuleFx
     private MessageSender messageSender;
 
+    @FXML
+    private Button saveButton;
+
 	public PersistentIdController(InRepositoryService inRepositoryService) {
 		this.inRepositoryService = inRepositoryService;
 	}
 
 	@FXML
     public void onSavePress(ActionEvent event) {
-        // todo
+        InRepositoryService.UpdateIdsResult result = inRepositoryService.updateIds(
+            selectedImageId,
+            persistentIdTextField.getText(),
+            parentPersistentIdTextField.getText(),
+            isVisibleForChildrenCheckBox.isSelected()
+        );
+        switch (result) {
+	        case InRepositoryService.UpdateIdsResult.Success success -> { } // what we wanted, so let's ignore it
+            case InRepositoryService.UpdateIdsResult.IdReuse idReuse -> {
+                Alerts.error("Persistent ID is already in use");
+            }
+        }
     }
 
     @FXML
@@ -72,6 +87,7 @@ public void onImageSelected(ImageSelectedEvent event) {
             internalIdTextField.setText(event.imageId().toString());
             internalIdTextField.setDisable(false);
             duplicateButton.setDisable(false);
+            saveButton.setDisable(false);
 
             Threading.onSeparateThread(keyStone, toolkit -> {
                 Ids id = inRepositoryService.getIds(event.imageId());
@@ -99,6 +115,7 @@ private void disable() {
         isVisibleForChildrenCheckBox.setSelected(false);
         parentPersistentIdTextField.setText("");
         persistentIdTextField.setText("");
+        saveButton.setDisable(true);
 
         internalIdTextField.setDisable(true);
         isVisibleForChildrenCheckBox.setDisable(true);

diff --git a/gui-local/src/main/resources/fx/repository/persistent_id_component.fxml b/gui-local/src/main/resources/fx/repository/persistent_id_component.fxml
@@ -43,7 +43,7 @@
       <FlowPane hgap="10.0" maxWidth="180.0" minWidth="140.0" prefWidth="180.0" vgap="10.0" GridPane.columnIndex="3">
          <children>
               <CheckBox fx:id="isVisibleForChildrenCheckBox" minWidth="15.0" mnemonicParsing="false" selected="true" text="Visible to children" />
-              <Button disable="true" mnemonicParsing="false" onAction="#onSavePress" text="Save" />
+              <Button fx:id="saveButton" disable="true" mnemonicParsing="false" onAction="#onSavePress" text="Save" />
             <Button fx:id="duplicateButton" disable="true" mnemonicParsing="false" onAction="#onDuplicatePress" text="Duplicate" />
          </children>
          <padding>