Merge pull request #162 from Sterc/feature-137

Adding checkbox allowing bulk redirect removal

Author: gpsietzema

assets/components/stercseo/js/mgr/widgets/redirects.grid.js

@@ -3,6 +3,8 @@ StercSEO.grid.Redirects = function(config) {
     if (!config.id) {
         config.id = 'stercseo-grid-redirects';
     }
+
+    this.sm = new Ext.grid.CheckboxSelectionModel();
     Ext.applyIf(config,{
         id: config.id
         ,url: StercSEO.config.connectorUrl
@@ -15,7 +17,8 @@ StercSEO.grid.Redirects = function(config) {
         ,autoHeight: true
         ,paging: true
         ,remoteSort: true
-        ,columns: [{
+        ,sm: this.sm
+        ,columns: [this.sm, {
             header: _('stercseo.uri_label')
             ,dataIndex: 'url'
             ,width: 280
@@ -37,6 +40,9 @@ StercSEO.grid.Redirects = function(config) {
             ,scope: this
             ,cls:'primary-button'
             ,id: 'btn-add-uri'
+        }, {
+            text: _('stercseo.uri_remove_bulk'),
+            handler: this.removeRedirectsFromGrid
         },'->',{
             xtype: 'modx-combo-context'
             ,fieldLabel: _('context')
@@ -154,6 +160,24 @@ Ext.extend(StercSEO.grid.Redirects,MODx.grid.Grid,{
         });
     }
 
+    ,removeRedirectsFromGrid: function () {
+        var cs = this.getSelectedAsList();
+        if (cs === false) return false;
+
+        MODx.msg.confirm({
+            title: _('stercseo.uri_remove_bulk'),
+            text: _('stercseo.uri_remove_bulk_confirm'),
+            url: this.config.url,
+            params: {
+                action: 'mgr/redirect/removeMultiple',
+                redirects: cs
+            },
+            listeners: {
+                'success': { fn:function() { this.refresh(); }, scope:this }
+            }
+        });
+    }
+
     ,filter: function (tf, nv, ov) {
         var store = this.getStore();
         var key = tf.getName();

core/components/stercseo/lexicon/de/default.inc.php

@@ -97,6 +97,10 @@
 $_lang['stercseo.uri_update'] = 'Weiterleitung bearbeiten';
 $_lang['stercseo.uri_remove'] = 'Weiterleitung löschen';
 $_lang['stercseo.uri_remove_confirm'] = 'Sind Sie sicher, dass Sie diese Weiterleitung löschen wollen?';
+$_lang['stercseo.uri_remove_bulk'] = 'Remove selected redirects';
+$_lang['stercseo.uri_remove_bulk_confirm'] = 'Are you sure you want to remove the selected redirects?';
+$_lang['stercseo.uri_remove_bulk.redirect_err_ns'] = 'Redirect not specified';
+
 $_lang['stercseo.migrate'] = 'Weiterleitung migrieren';
 $_lang['stercseo.migrate_desc'] = 'Sie können Ihre Weiterleitung (SEO Tab version 1.2.2 und niedriger) von Ressourcen Eigenschaften zu eigenständigen seoURL Objekten migireren. Dies wird automatisch die Daten aller Weiterleitungen für Sie migrieren. Es ist keine weitere Aktion notwendig. Bitte lassen Sie diese Seite geöffnet, damit SEO Tab die Migrierung korrekt beenden kann.';
 $_lang['stercseo.migrate_alert'] = 'Ihre SEO Tab Weiterleitungen müssen migriert werden, bitte klicken Sie hier um die Migrierungs Seite anzuzeigen.';

core/components/stercseo/lexicon/en/default.inc.php

@@ -96,6 +96,10 @@
 $_lang['stercseo.uri_update'] = 'Update redirect';
 $_lang['stercseo.uri_remove'] = 'Remove redirect';
 $_lang['stercseo.uri_remove_confirm'] = 'Are you sure you want to remove this redirect?';
+$_lang['stercseo.uri_remove_bulk'] = 'Remove selected redirects';
+$_lang['stercseo.uri_remove_bulk_confirm'] = 'Are you sure you want to remove the selected redirects?';
+$_lang['stercseo.uri_remove_bulk.redirect_err_ns'] = 'Redirect not specified';
+
 $_lang['stercseo.migrate'] = 'Migrate redirects';
 $_lang['stercseo.migrate_desc'] = 'Upgrading to 2.0.0 from 1.* is highly recommended, but it does introduce some risks. SEO Tab 1.* stored the 301 redirects into the properties-column of a resource. SEO Tab 2 stores it in a seperate database table. This means a migration is needed. The migration-process is memory-intensive and might take a very long time, so just let it roll until it mentions a finished migration.<br /><br />The migration is running while you read this. This page will automatically migrate all the redirects for you, so no action is required, but please keep this page open for SEO Tab to correctly handle the migration process.';
 $_lang['stercseo.migrate_alert'] = 'SEO Tab was updated, but your 301 redirects need to be migrated. Click here to start the migration.';

core/components/stercseo/lexicon/fr/default.inc.php

@@ -96,6 +96,10 @@
 $_lang['stercseo.uri_update'] = 'Mettre à jour';
 $_lang['stercseo.uri_remove'] = 'Supprimer';
 $_lang['stercseo.uri_remove_confirm'] = 'Êtes-vous sûr de vouloir supprimer cette redirection ?';
+$_lang['stercseo.uri_remove_bulk'] = 'Remove selected redirects';
+$_lang['stercseo.uri_remove_bulk_confirm'] = 'Are you sure you want to remove the selected redirects?';
+$_lang['stercseo.uri_remove_bulk.redirect_err_ns'] = 'Redirect not specified';
+
 $_lang['stercseo.migrate'] = 'Migrer les redirections';
 $_lang['stercseo.migrate_desc'] = 'Vous pouvez migrez vos redirections (des versions 1.2.2 et antérieures) des propriétés de ressources vers les objets seoUrl. Cette page migrera automatiquement les redirections, aucune action de votre part n\'est requise, mais veuillez garder cette page ouverte afin que SEO Tab puisse gérer correctement le processus de migration.';
 $_lang['stercseo.migrate_alert'] = 'Vos redirections ont besoin d\'être migrées. Cliquez ici pour vous rendre à la page de migration.';

core/components/stercseo/lexicon/nl/default.inc.php

@@ -96,6 +96,10 @@
 $_lang['stercseo.uri_update'] = 'Update redirect';
 $_lang['stercseo.uri_remove'] = 'Verwijder redirect';
 $_lang['stercseo.uri_remove_confirm'] = 'Weet je zeker dat je deze redirect wil verwijderen?';
+$_lang['stercseo.uri_remove_bulk'] = 'Verwijder geselecteerde redirects';
+$_lang['stercseo.uri_remove_bulk_confirm'] = 'Weet je zeker dat je de geselecteerde redirects wilt verwijderen?';
+$_lang['stercseo.uri_remove_bulk.redirect_err_ns'] = 'Redirect niet gespecificeerd';
+
 $_lang['stercseo.migrate'] = 'Migreer redirects';
 $_lang['stercseo.migrate_desc'] = 'Hier kun je je redirects (SEO Tab versie 1.2.2 en ouder) migreren van resource properties naar seoUrl objects. Deze pagina migreert al je redirects automatisch, een handeling is dus niet vereist. Houd a.u.b. wel deze pagina open zodat SEO Tab het migratieproces correct kan afhandelen.';
 $_lang['stercseo.migrate_alert'] = 'Je SEO Tab redirects moeten worden gemigreerd. Klik hier om de migratie te starten.';

core/components/stercseo/lexicon/ru/default.inc.php

@@ -96,6 +96,10 @@
 $_lang['stercseo.uri_update'] = 'Обновить редирект';
 $_lang['stercseo.uri_remove'] = 'Удалить редирект';
 $_lang['stercseo.uri_remove_confirm'] = 'Вы действительно хотите удалить этот редирект?';
+$_lang['stercseo.uri_remove_bulk'] = 'Remove selected redirects';
+$_lang['stercseo.uri_remove_bulk_confirm'] = 'Are you sure you want to remove the selected redirects?';
+$_lang['stercseo.uri_remove_bulk.redirect_err_ns'] = 'Redirect not specified';
+
 $_lang['stercseo.migrate'] = 'Миграция редиректа';
 $_lang['stercseo.migrate_desc'] = 'Здесь вы можете перенести свои редиректы (SEO Tab версии 1.2.2 и ниже) из свойств ресурса в объекты seoUrl. Эта страница будет автоматически переносить все перенаправления для вас, поэтому никаких действий не требуется, но пожалуйста, держите эту страницу открытой для SEO Tab, чтобы правильно обрабатывать процесс миграции.';
 $_lang['stercseo.migrate_alert'] = 'Редиректы вашей SEO-страницы необходимо перенести. Нажмите здесь, чтобы перейти на страницу перехода.';

core/components/stercseo/processors/mgr/redirect/removemultiple.class.php

@@ -0,0 +1,42 @@
+<?php
+include_once __DIR__ . '/remove.class.php';
+/**
+ * Remove multiple redirects
+ *
+ * @param string $redirects Comma-separated redirect ids
+ */
+
+class StercSeoRemoveMultipleProcessor extends StercSeoRemoveProcessor
+{
+    public $redirects = array();
+
+    public function initialize()
+    {
+        $redirects = $this->getProperty('redirects', '');
+        if (empty($redirects)) {
+            return $this->modx->lexicon('stercseo.uri_remove_bulk.redirect_err_ns');
+        }
+        $this->redirects = explode(',', $redirects);
+
+        return true;
+    }
+
+    public function process()
+    {
+        foreach ($this->redirects as $redirect) {
+            $this->setProperty('id', $redirect);
+            $initialized = parent::initialize();
+            if ($initialized === true) {
+                $process = parent::process();
+                if (!$process['success']) {
+                    return $process;
+                }
+            } else {
+                return $this->failure($initialized);
+            }
+        }
+        return $this->success();
+    }
+}
+
+return 'StercSeoRemoveMultipleProcessor';