Changeset 5909

Timestamp:

04/10/12 17:58:11 (12 years ago)

Author:

marcieli

Message:

Ticket #2593 - Concluída melhoria: não permitir marcadores com mesmo nome.

Location:

trunk/prototype/modules/mail

Files:

• interceptors/Helpers.php (modified) (1 diff)
• js/label.js (modified) (4 diffs)

trunk/prototype/modules/mail/interceptors/Helpers.php

@@ -307 +307 @@
                 $Labels = Controller::find(array('concept' => 'label'), false, array('filter' => array('i=', 'name', $nameLabel) ));
                 if($Labels){
-                        //Throw new Exception('Marcador existe');
-                        //$params['id'] = 'Marcador existe';
-                        return false;
+                        Throw new Exception('Já existe um marcador com esse nome.');
                 }
         }

trunk/prototype/modules/mail/js/label.js

@@ -115 +115 @@
                         DataLayer.commit(false, false, function(data){
                                 var hasLabel = true;
-                                        for(var i in data){
-                                                if(parseInt(data[i].id))
-                                                        hasLabel = false;
+                                $.each(data, function(index, value) {
+                                        if(typeof value == 'object'){
+                                                hasLabel = false;
                                         }
+                                        else {
+                                                hasLabel = value;
+                                        }
+                                });
+                                        
+                                if(typeof data[0] == 'object'){
+                                        hasLabel = false;
+                                }
                                         if(!hasLabel){
                                                 $(".label-list-container .label-list li").each(function(){
@@ -162 +170 @@
                                                 loadMenu();
                                         }else{
-                                                alert('Problema ao salvar marcador.');
+                                                alert(hasLabel);
                                         }
                                         
@@ -244 +252 @@
         
         DataLayer.commit(false, false, function(data){
-        var hasLabel = true;
-                for(var i in data){
-                        if(parseInt(data[i].id))
+                var hasLabel = true;
+                $.each(data, function(index, value) {
+                        if(typeof value == 'object'){
                                 hasLabel = false;
-                }
+                        }else{
+                                hasLabel = value;
+                        }
+                });
                 if(!hasLabel){
                         labels = DataLayer.get('label',{criteria:{order: 'id'}}, true);
@@ -303 +314 @@
                         $(".add-label-button").empty()
                         .addClass("ui-icon ui-icon-plus");
-                        alert('Problema ao salvar marcador.');
+                        alert(hasLabel);
                 }
         });