Dotclear

Ticket #999 (closed idea: fixed)

Opened 15 years ago

Last modified 12 years ago

Le plugin Maintenance n'est pas extensible

Reported by: Moe Owned by: JcDenis
Priority: normal Milestone: 2.6
Component: refonte:dev Version: 2.1
Severity: normal Keywords:
Cc:

Description

Le plugin Maintenance ne propose pas de behaviour pour ajouter des boutons sur sa page.

Le plugin  Static Cache pourrait profiter d'un tel behaviour pour proposer un bouton "effacer le cache de Static Cache". Cela éviterait de créer une page d'administration spécialement pour ce plugin.

Change History

comment:1 Changed 15 years ago by Osku

Excellente idée. Beaucoup de plugins pourraient en user et en abuser :)

comment:2 Changed 15 years ago by Moe

On peut plus attacher de diff en pièce jointe ? Je le propose ici plutôt que de commiter sur le SVN du bazar, parce qu'il y a des points à discuter : pertinence de ces ajouts ? noms des behaviors ? emplacements de ceux-ci ?

J'ai choisi d'utiliser le préfixe "plugin" pour les behaviors, il n'existe pas encore :  http://fr.dotclear.org/documentation/2.0/resources/plugins/behaviors

Index: plugins/maintenance/index.php
===================================================================
--- plugins/maintenance/index.php	(révision 3138)
+++ plugins/maintenance/index.php	(copie de travail)
@@ -61,6 +61,9 @@
 	}
 }
 
+# --BEHAVIOR-- pluginMaintenanceHeaders
+$core->callBehavior('pluginMaintenanceHeaders',$core,$action,$start);
+
 ?>
 <html>
 <head>
@@ -84,6 +87,9 @@
 	echo '<p class="message">'.__('Logs deleted.').'</p>';
 }
 
+# --BEHAVIOR-- pluginMaintenanceMessage
+$core->callBehavior('pluginMaintenanceMessage',$core);
+
 if ($action == 'index' && !empty($_GET['indexposts']))
 {
 	$limit = 1000;
@@ -157,6 +163,7 @@
 	'<form action="plugin.php" method="get">'.
 	'<p><input type="submit" name="indexposts" value="'.__('Index all posts').'" /> '.
 	'<input type="submit" name="indexcomments" value="'.__('Index all comments').'" /> '.
+	$core->formNonce().
 	form::hidden(array('action'),'index').
 	form::hidden(array('p'),'maintenance').'</p>'.
 	'</form>';
@@ -178,6 +185,9 @@
 	form::hidden(array('action'),'empty_cache').
 	form::hidden(array('p'),'maintenance').'</p>'.
 	'</form>';
+	
+	# --BEHAVIOR-- pluginMaintenanceForm
+	$core->callBehavior('pluginMaintenanceForm',$core);
 }
 ?>
 

comment:3 Changed 15 years ago by xave

Je suis contre le "pluginMaintenanceMessage". Il est préférable de construire les message avant l'affichage de la page et de prévoir un affichage générique. C'est comme ça que font les pages de l'admin.

comment:4 Changed 15 years ago by xave

  • Milestone changed from 2.2 to 2.3

comment:5 Changed 15 years ago by kozlika

  • Owner team deleted
  • Component changed from module:ergo to refonte:dev
  • Milestone changed from 2.3 to Refonte admin

A discuter entre ergo et dev

comment:6 Changed 14 years ago by Tomtom33

Est-ce que l'on peut prioriser la réflexion sur ce ticket au niveau ergo? Parce que sa résolution (facile en plus) permettrai de pouvoir régler le ticket #1016 dans la foulé

comment:8 Changed 14 years ago by bruno

  • Priority changed from low to normal

comment:9 in reply to: ↑ 7 Changed 14 years ago by bruno

Replying to Moe:

Concernant le plugin Maintenance :  https://bitbucket.org/n1/dotclear/diff/plugins/maintenance/index.php?diff2=7238176d70cb&diff1=120c4648cb6f

Moe, j'ai peur qu'avec une évolution comme ça, on se retrouve avec une page pleine de boutons sans réelle cohérence. Je pense qu'il faudrait d'abord redesigner un peu la structure de cette page pour identifier différentes sections, avec pourquoi pas des onglets supplémentaires.

comment:10 Changed 13 years ago by franck

  • Status changed from new to onhold

C'est vrai que le plugin Maintenance n'est pas très sexy/explicite/extensible (qui a dit que c'était pas le seul ?). À revoir donc.

L'idée d'avoir des onglets me plait pas mal (genre Base de données, Cache, Système, …).

comment:11 Changed 12 years ago by franck

  • Milestone changed from Refonte admin to 2.6

comment:12 Changed 12 years ago by franck

  • Owner set to JcDenis
  • Status changed from onhold to new

comment:14 Changed 12 years ago by Denis Jean-Chirstian <contact@…>

(In [567a4a03e5df]) Revamp plugin maintenance, step 1, rewrite code to open it, addresses #999

comment:15 Changed 12 years ago by Denis Jean-Chirstian <contact@…>

(In [0fb7e85337bf]) Revamp plugin maintenance, step 2, add task reminder, addresses #999

comment:16 Changed 12 years ago by Denis Jean-Chirstian <contact@…>

(In [b0bef03695c0]) Revamp plugin maintenance, step 3, add backup tasks, dynamic tabs and plugin importExport, addresses #999

comment:17 Changed 12 years ago by brol

Attention : les messages ne sont pas à la bonne place, il faut les mettre après h2 breadcrumb.

De plus il manque un [ après CDATA à la ligne 75 du fichier \dotclear\plugins\maintenance\index.php :

	"//<![CDATA\n".
Last edited 12 years ago by brol (previous) (diff)

comment:18 Changed 12 years ago by Denis Jean-Chirstian <contact@…>

(In [2c77682177b4]) Plugin maintenance, small fixes and boss requests, addresses #999

comment:19 Changed 12 years ago by Denis Jean-Chirstian <contact@…>

(In [d2ef655d3195]) Revamp plugin maintenance, step 4, add user prefs (without translation), wait for boss, addresses #999

comment:20 Changed 12 years ago by Denis Jean-Chirstian <contact@…>

(In [0b0cb9cd7da7]) Revamp plugin maintenance, step 4 bis, better place for settings according to boss and some typo, addresses #999

comment:21 Changed 12 years ago by Denis Jean-Chirstian <contact@…>

(In [b234959829da]) Revamp plugin maintenance, step 5, now open to admin user and various fixes, addresses #999

comment:22 Changed 12 years ago by Denis Jean-Chirstian <contact@…>

  • Status changed from new to closed
  • Resolution set to fixed

(In [4a3330bc8bd5]) Revamp plugin maintenance, final setp (perhaps still some typo), addresses #1484, fixes #1208, fixes #999

Note: See TracTickets for help on using tickets.

Sites map