Revert revision
Eingetragen von sewimoeller (27)am 21.10.2010 - 17:11 Uhr in
Hallo zusammen,
eine Frage zu Versionierung in Drupal. Der Node-Modul ruft den Hook hook_nodeapi() mit op='delete revision' wenn eine Version gelöscht wird. Das funktioniert, alles gut. Nur wenn ein Node auf eine alte Version zurückgesetzt wird, erwarte ich dass das selbe passiert, nur op='revert revision' ist. Das ist aber nicht der Fall.
Im Code von Node-Modul wird gar nix gemacht, wenn ein Node auf die eine alte Version zurückgesetzt wird. Hier ist der Code von Node-Modul, der das macht:
function node_revision_delete_confirm_submit($form, &$form_state) {
$node_revision = $form['#node_revision'];
db_query("DELETE FROM {node_revisions} WHERE nid = %d AND vid = %d", $node_revision->nid, $node_revision->vid);
node_invoke_nodeapi($node_revision, 'delete revision');
watchdog('content', '@type: deleted %title revision %revision.', array('@type' => $node_revision->type, '%title' => $node_revision->title, '%revision' => $node_revision->vid));
drupal_set_message(t('Revision from %revision-date of @type %title has been deleted.', array('%revision-date' => format_date($node_revision->revision_timestamp), '@type' => node_get_types('name', $node_revision), '%title' => $node_revision->title)));
$form_state['redirect'] = 'node/'. $node_revision->nid;
if (db_result(db_query('SELECT COUNT(vid) FROM {node_revisions} WHERE nid = %d', $node_revision->nid)) > 1) {
$form_state['redirect'] .= '/revisions';
}Also bei delete wird node_invoke_nodeapi() aufgerufen. Hier ist der Code für revert:
function node_revision_revert_confirm_submit($form, &$form_state) {
$node_revision = $form['#node_revision'];
$node_revision->revision = 1;
$node_revision->log = t('Copy of the revision from %date.', array('%date' => format_date($node_revision->revision_timestamp)));
if (module_exists('taxonomy')) {
$node_revision->taxonomy = array_keys($node_revision->taxonomy);
}
node_save($node_revision);
watchdog('content', '@type: reverted %title revision %revision.', array('@type' => $node_revision->type, '%title' => $node_revision->title, '%revision' => $node_revision->vid));
drupal_set_message(t('@type %title has been reverted back to the revision from %revision-date.', array('@type' => node_get_types('name', $node_revision), '%title' => $node_revision->title, '%revision-date' => format_date($node_revision->revision_timestamp))));
$form_state['redirect'] = 'node/'. $node_revision->nid .'/revisions';
}Beide Funktionen sind in node.pages.inc zu finden.
Die Frage ist jetzt, ist das ein Design-Fehler von Drupal, dass bei Revert keine Hooks aufgerufen werden oder verstehe ich was falsch? Bei meiner Implementierung fehlt mir nur noch ein Hook für Versionen zurücksetzen. Weiß jemand Bescheid? Vielen Dank im voraus.
- Anmelden oder Registrieren um Kommentare zu schreiben
patch für 'revert revision'
am 21.10.2010 - 17:34 Uhr
Hab noch eine Diskussion zu dem Thema gefunden:
http://drupal.org/node/524726
Sie haben leider kein Patch bereitgestellt, der für 6.x Version einen neuen op='revert revision' für hook_nodeapi bereitstellt. Ich weiss nicht, wie man einen Patch erstellt... keine Ahnung wie ich jetzt vorgehen soll...
hallo zusammen, habe gestern
am 22.10.2010 - 12:43 Uhr
hallo zusammen,
habe gestern festgestellt, dass alle Daten, die mit dem Hook hook_load() geladen werden, werden richtig aktualisiert und das Zurücksetzen der Version funktioniert richtig. Aber ich möchte auch andere Tabellen "versionsfähig" machen. D.h. ich muss dann alle Daten in der Funktion my_module_load() laden. Das können unter Umständen sehr viele sein. Und ich denke der Hook hook_load() ist nicht dafür gedacht, sondern um den Node zu laden... oder doch? Sollen alle Daten, die auf irgendwelche Art und Weise mit dem Node verbunden sind, hier geladen werden?
Für eure Tipps werde ich mich sehr freuen!