Changeset 18484

Timestamp:

Oct 5, 2012, 9:36:38 PM (12 years ago)

Author:

rvelices

Message:

added "are you sure?" confirmation before deleting all derivatives in maintenance or before deleting more than 500 derivatives in the batch manager

Location:

trunk/admin/themes/default/template

Files:

• batch_manager_global.tpl (modified) (7 diffs)
• maintenance.tpl (modified) (1 diff)

trunk/admin/themes/default/template/batch_manager_global.tpl

@@ -95 +95 @@
 var nb_thumbs_page = {$nb_thumbs_page};
 var nb_thumbs_set = {$nb_thumbs_set};
+var are_you_sure = "{'Are you sure?'|@translate|@escape:'javascript'}";
 var applyOnDetails_pattern = "{'on the %d selected photos'|@translate}";
 var all_elements = [{if !empty($all_elements)}{','|@implode:$all_elements}{/if}];
@@ -188 +189 @@
     }
     else {
-      $(".thumbnails input[type=checkbox]").each(function() {
-         if ($(this).is(':checked')) {
-           nbSelected++;
-         }
-      });
+      nbSelected = $(".thumbnails input[type=checkbox]").filter(':checked').length;
     }
 
@@ -418 +415 @@
 
   jQuery('#applyAction').click(function() {
-                if (jQuery('[name="selectAction"]').val() != 'generate_derivatives'
+                var action = jQuery('[name="selectAction"]').val();
+                if (action == 'delete_derivatives') {
+                        var d_count = $('#action_delete_derivatives input[type=checkbox]').filter(':checked').length
+                                , e_count = $('input[name="setSelected"]').is(':checked') ? nb_thumbs_set : $('.thumbnails input[type=checkbox]').filter(':checked').length;
+                        if (d_count*e_count > 500)
+                                return confirm(are_you_sure);
+                }
+                
+                if (action != 'generate_derivatives'
                         || derivatives.finished() )
                 {
@@ -436 +441 @@
                         derivatives.elements = all_elements;
                 else
-                        jQuery('input[name="selection[]"]').each(function() {
+                        jQuery('.thumbnails input[type=checkbox]').each(function() {
                                 if (jQuery(this).attr('checked')) {
                                         derivatives.elements.push(jQuery(this).val());
@@ -581 +586 @@
         <option value="filter_dimension" {if isset($filter.dimension)}disabled="disabled"{/if}>{'Dimensions'|@translate}</option>
       </select>
-<!--      <input id="removeFilters" class="submit" type="submit" value="Remove all filters" name="removeFilters"> -->
       <a id="removeFilters" href="">{'Remove all filters'|@translate}</a>
     </p>
@@ -713 +717 @@
             {html_options options=$associate_options }
          </select>
-<br>{'... or '|@translate}</span><a href="#" class="addAlbumOpen" title="{'create a new album'|@translate}">{'create a new album'|@translate}</a>
+<br>{'... or '|@translate} <a href="#" class="addAlbumOpen" title="{'create a new album'|@translate}">{'create a new album'|@translate}</a>
     </div>
 
@@ -721 +725 @@
             {html_options options=$move_options }
          </select>
-<br>{'... or '|@translate}</span><a href="#" class="addAlbumOpen" title="{'create a new album'|@translate}">{'create a new album'|@translate}</a>
+<br>{'... or '|@translate} <a href="#" class="addAlbumOpen" title="{'create a new album'|@translate}">{'create a new album'|@translate}</a>
     </div>
 

trunk/admin/themes/default/template/maintenance.tpl

@@ -36 +36 @@
         <li><a href="{$U_MAINT_COMPILED_TEMPLATES}">{'Purge compiled templates'|@translate}</a></li>
         <li>{'Delete multiple size images'|@translate}: 
-        {foreach from=$purge_derivatives key=name item=url name=loop}{if !$smarty.foreach.loop.first}, {/if}<a href="{$url}">{$name}</a>{/foreach}
+        {foreach from=$purge_derivatives key=name item=url name=loop}{if !$smarty.foreach.loop.first}, {/if}<a href="{$url}"{if $smarty.foreach.loop.first} onclick="return confirm('{'Are you sure?'|@translate|@escape:javascript}');"{/if}>{$name}</a>{/foreach}
         </li>
 </ul>