Prevent changes to created_by and version tags on changesets

diff --git a/TODO.txt b/TODO.txt
index f1cb4fa58ab14bb9f89739501ba5fe4fc902430d..84262975a3001ded94a9f854fc84688ffb05808e 100644 (file)
--- a/TODO.txt
+++ b/TODO.txt
@@ -19,7 +19,6 @@ Potlatch 2: main outstanding issues
 == Saving ==
 
 * Should be able to reuse changesets
 == Saving ==
 
 * Should be able to reuse changesets

-* Shouldn't be able to change created_by and version on changeset

 
 == Tag editing ==
 
 
 == Tag editing ==
 

diff --git a/net/systemeD/potlatch2/save/SaveDialog.mxml b/net/systemeD/potlatch2/save/SaveDialog.mxml
index a3fa522e9089941eeacc5aa3c0b7b28c95869d02..c8ab631ece929532a7c889d84cb5f04422a4ce9d 100644 (file)
--- a/net/systemeD/potlatch2/save/SaveDialog.mxml
+++ b/net/systemeD/potlatch2/save/SaveDialog.mxml
@@ -30,7 +30,9 @@
         <mx:VBox width="100%" height="100%" label="Advanced">
           <mx:Label text="Changeset tags:"/>
           <mx:DataGrid editable="true" width="100%" id="advancedTagGrid"
         <mx:VBox width="100%" height="100%" label="Advanced">
           <mx:Label text="Changeset tags:"/>
           <mx:DataGrid editable="true" width="100%" id="advancedTagGrid"

-              dataProvider="{changesetTags}">
+              dataProvider="{changesetTags}"
+              itemEditBeginning="allowForEdit(event)"
+              itemEditEnd="verifyInput(event)">

             <mx:columns>
                 <mx:DataGridColumn editable="true" dataField="k" headerText="Key"/>
                 <mx:DataGridColumn editable="true" dataField="v" headerText="Value"/>
             <mx:columns>
                 <mx:DataGridColumn editable="true" dataField="k" headerText="Key"/>
                 <mx:DataGridColumn editable="true" dataField="v" headerText="Value"/>


@@ -75,6 +77,8 @@
     import mx.controls.*;
     import mx.managers.PopUpManager;
        import mx.core.Application;
     import mx.controls.*;
     import mx.managers.PopUpManager;
        import mx.core.Application;

+    import mx.events.DataGridEvent;
+    import mx.events.DataGridEventReason;

     
     import net.systemeD.halcyon.connection.*;
     
     
     import net.systemeD.halcyon.connection.*;
     


@@ -116,6 +120,28 @@
         conn.createChangeset(tags);
     }
 
         conn.createChangeset(tags);
     }
 

+    private function allowForEdit(event:DataGridEvent):void {
+        /* check before editing the tag grid that it's neither created_by nor version tags */
+        var item:Object = ((event.currentTarget as DataGrid).dataProvider as ArrayCollection)[event.rowIndex];
+        if(item.k == 'created_by' || item.k == 'version') {
+          event.preventDefault();
+        }
+    }
+
+    private function verifyInput(event:DataGridEvent):void {
+        /* check before saving any change that the new key isn't created_by nor version */
+        // it is OK if the user cancels the edit
+        if( event.reason == DataGridEventReason.CANCELLED ) return;
+
+        var editor:TextInput = (event.currentTarget as DataGrid).itemEditorInstance as TextInput;
+
+        if( event.dataField == "k" ) {
+          if( editor.text == 'created_by' || editor.text == 'version' ) {
+            event.preventDefault();
+          }
+        }
+    }
+

     private function changesetCreated(event:EntityEvent):void {
         var changeset:Changeset = conn.getActiveChangeset();
         addStatus("Changeset created (id: "+changeset.id+")");
     private function changesetCreated(event:EntityEvent):void {
         var changeset:Changeset = conn.getActiveChangeset();
         addStatus("Changeset created (id: "+changeset.id+")");