Changeset 1896

Timestamp:

04/20/12 13:22:08 (12 years ago)

Author:

twagoo

Message:

Linked delete button on comments to delete service. Extended DeleteService? to allow deletion of comments in addition to items. Extended Comment to provide required information. Refs #185

Location:

ComponentRegistry/trunk/ComponentBrowserGui/src/main/flex/clarin/cmdi/componentregistry

Files:

• browser/CommentPanel.as (modified) (2 diffs)
• browser/CommentsPanel.as (modified) (3 diffs)
• common/Comment.as (modified) (4 diffs)
• services/CommentListService.as (modified) (3 diffs)
• services/DeleteService.as (modified) (2 diffs)

ComponentRegistry/trunk/ComponentBrowserGui/src/main/flex/clarin/cmdi/componentregistry/browser/CommentPanel.as

@@ -2 +2 @@
 {
         import clarin.cmdi.componentregistry.common.Comment;
+        import clarin.cmdi.componentregistry.services.DeleteService;
         
         import flash.events.MouseEvent;
@@ -74 +75 @@
                 private function deleteHandler(event:CloseEvent):void {
                         if(event.detail == Alert.OK){
-                                Alert.show("Will delete","TODO");
+                                DeleteService.instance.deleteComment(comment);
                         }       
                 }

ComponentRegistry/trunk/ComponentBrowserGui/src/main/flex/clarin/cmdi/componentregistry/browser/CommentsPanel.as

@@ -6 +6 @@
         import clarin.cmdi.componentregistry.services.CommentListService;
         import clarin.cmdi.componentregistry.services.CommentPostService;
+        import clarin.cmdi.componentregistry.services.DeleteService;
         
         import flash.events.Event;
@@ -36 +37 @@
                         this.setStyle("paddingTop", 5);
                         this.setStyle("paddingBottom", 5);
+                        
+                        // this is for responding to the deletion of comments. At this point there is no way to distinghuish between item and component deletion
+                        // and that probably is fine since they mostly require the same response. It does mean that this component will also reload when a
+                        // component gets deleted, which is a bit superfluous.
+                        DeleteService.instance.addEventListener(DeleteService.ITEM_DELETED, commentDeletedHandler);
                 }
                 
@@ -94 +100 @@
                         load();
                 }
+                
+                private function commentDeletedHandler(event:Event):void {
+                        load();
+                }
         }
 }

ComponentRegistry/trunk/ComponentBrowserGui/src/main/flex/clarin/cmdi/componentregistry/common/Comment.as

@@ -1 +1 @@
 package clarin.cmdi.componentregistry.common
 {
+        import clarin.cmdi.componentregistry.services.Config;
+
         public class Comment implements XmlAble
         {
@@ -16 +18 @@
                 public var id:String;
                 public var canDelete:Boolean;
+                public var itemDescription:ItemDescription;
+                public var dataUrl:String;
                 
                 public function Comment():void{
@@ -21 +25 @@
                 }
                 
-                public function create(comment:XML):void{
+                public function create(comment:XML, itemDescription:ItemDescription):void{
                         this.comments = comment.comments;
                         this.commentDate  = DateUtils.formatDateString(comment.commentDate);
@@ -29 +33 @@
                         this.userName = comment.userName;
                         this.canDelete = comment.canDelete == "true";
+                        
+                        this.itemDescription = itemDescription;
+                        if(itemDescription.isProfile){
+                                dataUrl = Config.instance.getProfileCommentsPath(itemDescription.id) + id;
+                        } else {
+                                dataUrl = Config.instance.getComponentCommentsPath(itemDescription.id) + id;
+                        }
                 }
                 

ComponentRegistry/trunk/ComponentBrowserGui/src/main/flex/clarin/cmdi/componentregistry/services/CommentListService.as

@@ -13 +13 @@
                 ;
                 private var userSpace:Boolean;
+                private var itemDescription:ItemDescription;
                 
                 /**
@@ -30 +31 @@
                         super(url);
                         
+                        this.itemDescription = itemDescription;
                         this.userSpace = userSpace;
                 }
@@ -46 +48 @@
                         for each (var node:XML in nodes) {
                                 var comment:Comment = new Comment();
-                                comment.create(node);
+                                comment.create(node, itemDescription);
                                 comments.addItem(comment);
                         }

ComponentRegistry/trunk/ComponentBrowserGui/src/main/flex/clarin/cmdi/componentregistry/services/DeleteService.as

@@ -1 +1 @@
 package clarin.cmdi.componentregistry.services {
+        import clarin.cmdi.componentregistry.common.Comment;
         import clarin.cmdi.componentregistry.common.ItemDescription;
-
+        
         import com.adobe.net.URI;
-
+        
         import flash.events.Event;
         import flash.events.EventDispatcher;
-
+        
         import mx.controls.Alert;
         import mx.managers.CursorManager;
@@ -40 +41 @@
                                 url.setQueryValue(Config.PARAM_USERSPACE, "true");
                         }
+                        sendDelete(url);
+                }
+                
+                public function deleteComment(comment:Comment):void {
+                        // Deletion of comments triggers the same response as deletion of items (which was there first). This is suboptimal
+                        // but will do for now. E.g. the error messages refer to items but that is generic enough to work in the context of comment deletion. 
+                        // Also there is just one event, splitting this out is trivial but a bit messy.
+                        
+                        CursorManager.setBusyCursor();
+                        var url:URI = new URI(comment.dataUrl);
+                        if (comment.itemDescription.isInUserSpace) {
+                                url.setQueryValue(Config.PARAM_USERSPACE, "true");
+                        }
+                        sendDelete(url);
+                }
+                
+                private function sendDelete(url:URI):void {
                         service.url = url.toString();
                         service.send(DELETE_METHOD);