Changeset 2592

Timestamp:

02/14/13 20:02:54 (11 years ago)

Author:

olhsha

Message:

null-pointer exception after deletening a comment is semi-fixed: that is no null-pointer exception because there are checking "if(_itemDescription)" in two places. If it is the case then nothing happens. However, the reason why _itemDescription gets forgotten after deleating a comment is unclear. In principle can be deployed after testing, but just it will not be that convenient for a user, if after each comment delete the infor-panel gets totally refreshed.

Location:

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

Files:

• CommentsPanel.as (modified) (1 diff)
• infoPage.mxml (modified) (1 diff)

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

@@ -125 +125 @@
                         if(service) {
                                 var commentsCount:int = service.comments.length;
-                                _itemDescription.commentsCount = commentsCount;
+                                
+                                if (_itemDescription != null) {_itemDescription.commentsCount = commentsCount;}
                                 
                                 if(commentsCount > 0) {

ComponentRegistry/trunk/ComponentBrowserGui/src/main/flex/clarin/cmdi/componentregistry/browser/infoPage.mxml

@@ -75 +75 @@
                         
                         private function get commentsLabel():String{
+                                if (_itemDescription == null) {return "info panel has been reloaded";};
                                 return "Comments (" + _itemDescription.commentsCount + ")";
                         }