history rework #1502

bitshiftersgmbh · 2024-04-24T09:47:33Z

complex rework of yform manager history views, both list and details view

adding missing and new translations in 4 languages (TODO: pt_br)
comparison based on field type (complex diff with <del> and <ins> for text/textarea values)
for technical reasons (comparison with prev revision) changed sorting to timestamp ASC
details modal view with icons, tooltips (file type) and grouping changes/added/removed/unchanged into accordion panels

(reverting const types, since only php 8.3+ | bringing back else area |

skerbis · 2024-04-26T14:46:44Z

Bekomme aktuell beim Aufruf einer einfachen Tabelle:

Error: Call to a member function hasValue() on null
File: redaxo/src/addons/yform/plugins/manager/pages/data_history.php
Line: 304

Stacktrace

Function	File	Line
rex_yform_manager::{closure}	redaxo/src/core/lib/util/formatter.php	494
rex_formatter::custom	redaxo/src/core/lib/util/formatter.php	31
rex_formatter::format	redaxo/src/core/lib/list.php	1147
rex_list->formatValue	redaxo/src/core/lib/list.php	526
rex_list->getColumnValue	redaxo/src/core/lib/list.php	1328
rex_list->get	redaxo/src/addons/yform/plugins/manager/pages/data_history.php	372
include	redaxo/src/addons/yform/plugins/manager/lib/yform/manager.php	212
rex_yform_manager->getDataPage	redaxo/src/addons/yform/plugins/manager/pages/data_edit.php	20
require	redaxo/src/core/lib/packages/package.php	233
rex_package->includeFile	redaxo/src/core/lib/be/controller.php	508
rex_be_controller::{closure}	redaxo/src/core/lib/util/timer.php	56
rex_timer::measure	redaxo/src/core/lib/be/controller.php	490
rex_be_controller::includePath	redaxo/src/core/lib/be/controller.php	456
rex_be_controller::includeCurrentPageSubPath	redaxo/src/addons/yform/pages/index.php	3
require	redaxo/src/core/lib/packages/package.php	233
rex_package->includeFile	redaxo/src/core/lib/be/controller.php	508
rex_be_controller::{closure}	redaxo/src/core/lib/util/timer.php	56
rex_timer::measure	redaxo/src/core/lib/be/controller.php	490
rex_be_controller::includePath	redaxo/src/core/lib/be/controller.php	439
rex_be_controller::includeCurrentPage	redaxo/src/core/backend.php	253
require	redaxo/src/core/boot.php	157
require	redaxo/index.php	9

System report (REDAXO 5.17.1, PHP 8.2.18, MariaDB 10.11.7)

REDAXO
Version	5.17.1

PHP
Version	8.2.18
OPcache	yes
Xdebug	no

Database
Version	MariaDB 10.11.7
Character set	utf8mb4

Server
OS	Linux
SAPI	fpm-fcgi
Webserver	Apache

Request
Browser	Safari/17.4.1
Protocol	HTTP/1.0
HTTPS	yes

Packages
adminer	2.0.1
articletemplates	1.0.1
be_style	3.4.1
be_style/customizer	3.3.1
be_style/redaxo	3.4.1
bloecks	4.0.2
bloecks/cutncopy	4.0.2
bloecks/dragndrop	4.0.2
cke5	6.1.0
cropper	2.0.3
fa_iconpicker	1.2.0
for_sa11y	4.1.0
forcal	4.0.0
icecoder	3.0.2
install	2.12.1
maintenance	2.9.2
mblock	3.4.13
media_manager	2.16.0
mediapool	2.14.0
metainfo	2.11.0
mform	8.0.0-beta3
minibar	2.4.3
navigation_array	3.0.0
phpmailer	2.13.1
project	dev
quick_navigation	8.0.0-dev
redactor	2.4.2
rex_sql_helper	1.0.0-beta1
structure	2.17.1
structure/content	2.17.1
structure/history	2.17.0
terminal	0.0.1
uploader	2.4.2
users	2.11.0
watson	2.3.0
ycom	4.3.0
ycom/auth	4.3.0
ycom/group	4.3.0
ycom/media_auth	4.3.0
yform	4.2.1
yform/email	4.2.1
yform/manager	4.2.1
yform_spam_protection	1.2.4
yrewrite	2.10.0
zip_install	1.4.0

bitshiftersgmbh · 2024-04-26T15:05:46Z

@skerbis guck ich mir Sonntag an, ok? Melde mich ggf. im Slack kommende Woche wenn ich Zuarbeit brauche. 👍🏻

…does not result in an existing dataset rex stan: more fixes

bitshiftersgmbh · 2024-04-28T15:20:26Z

@skerbis Der Fehler taucht(e) auf, wenn keine oder keine in ein existierendes Dataset aufklösende dataset_id als URL Param übergeben wurde. Ich habe es jetzt (erst einmal) so gelöst, dass dann ein Warning erscheint und die 2 Spalten "ansehen" und "Vergleich Änderungen mit aktueller Version" dann nicht ausgegeben werden. Check mal bitte ...

skerbis · 2024-04-29T06:58:29Z

Super. 👍🏻

dergel · 2024-05-20T20:27:39Z

plugins/manager/fragments/yform/manager/history.diff.php

+        '</a>
+        <button type="button" class="btn btn-default" data-dismiss="modal" aria-hidden="true">&times;</button>
+    </div>
+    <script>


Hier müsste noch nonce="' . rex_response::getNonce() . '" noch rein, aber lieber noch, wenn das ausgelagert wird, in die yform js Dateien, oder ein eigenes für historie anlegen

Ich leg ne .js an

dergel · 2024-05-20T20:28:13Z

plugins/manager/fragments/yform/manager/history.diff.php

+$content .= '
+    </div>
+    <div class="modal-footer">
+        <a href="index.php?page=yform/manager/data_edit?' . $restoreUrl . '" class="btn btn-warning" onclick="return confirm(\'' . rex_i18n::msg('yform_history_restore_confirm') . '\')">' .


kein onclick bitte, sondern js in Dateien auslagern. CSP macht da sonst Probleme

wird gemacht

dergel · 2024-05-20T20:28:47Z

finde ich ziemlich cool :)

Beispielaufruf bei einem Datensatz:

Warning: Undefined array key "mysinglewidget" in [redaxo/src/addons/yform/plugins/manager/pages/data_history.php] on line 365
Warning: Undefined array key "mymultiplewidget" in [redaxo/src/addons/yform/plugins/manager/pages/data_history.php] on line 365
Warning: Undefined array key "group_id" in [redaxo/src/addons/yform/plugins/manager/pages/data_history.php] on line 365
Warning: Undefined array key "pool" in [redaxo/src/addons/yform/plugins/manager/pages/data_history.php] on line 365
Warning: Undefined array key "media_single" in [redaxo/src/addons/yform/plugins/manager/pages/data_history.php] on line 365
Warning: Undefined array key "media_multiple" in [redaxo/src/addons/yform/plugins/manager/pages/data_history.php] on line 365

und bei einem DIFF Aufruf

Warning: Undefined array key "deleted" in [redaxo/src/addons/yform/plugins/manager/fragments/yform/manager/history.diff.php] on line 65
Warning: Undefined array key "count" in [redaxo/src/addons/yform/plugins/manager/fragments/yform/manager/history.diff.php] on line 65
Warning: Undefined array key "rows" in [redaxo/src/addons/yform/plugins/manager/fragments/yform/manager/history.diff.php] on line 120
Warning: Undefined array key "icon" in [redaxo/src/addons/yform/plugins/manager/fragments/yform/manager/history.diff.php] on line 165

Irgendwie geht die Historysuche nicht.

bitshiftersgmbh · 2024-05-20T20:37:44Z

@dergel check ich alles Ende der Woche, maybe erst Anfang kommende ... 🙃

bitshiftersgmbh · 2024-06-07T15:02:53Z

@dergel

Beispielaufruf bei einem Datensatz:

Kannst du mir hier n Abzug organisieren (leere Table + zugehörige History-Daten), bitte?

und bei einem DIFF Aufruf

ist behoben (Naming Fehler, Rest Folgefehler)

bitshiftersgmbh · 2024-06-07T15:31:40Z

@skerbis guck ich mir Sonntag an, ok? Melde mich ggf. im Slack kommende Woche wenn ich Zuarbeit brauche. 👍🏻

Ach ich bin doof. Es gibt noch diesen "Historie" Button oben. Da wird natürlich keine ID übergeben. Ich bin immer nur über den Button im Funktionen-Menubutton gegangen (also pro Datensatz).

now old history is present when table history is opened and new more functional history on dataset specific history + bringing back search + marker for current dataset in dataset specific mode + added css + outsourced js

dergel · 2024-06-20T10:13:06Z

plugins/manager/pages/data_history.php

+                $prevHistoryDataset = $historyDatasets[$rev - 1]['values'];
+
+                foreach ($historyDataset as $field => $value) {
+                    if ('' . $historyDataset[$field] !== '' . $prevHistoryDataset[$field]) {


Es gibt hier undefined array bei mir. das müsste man wohl noch abfangen

dergel · 2024-06-20T10:15:31Z

plugins/manager/pages/data_history.php


 $list = rex_list::factory($listQuery, defaultSort: [
+    'timestamp' => 'asc',


lieber desc, dann hat man immer die neuesten oben und kann direkt vergleichen, das war aber vorher schon "falsch" :)

dergel · 2024-06-20T10:50:23Z

wie besprochen gibt noch die Problematik, dass man hier davon ausgeht, dass alle Einträge in einer Liste sind und es daher beim Paging nicht 100%ig funktioniert. Deswegen ist eine Sortierung bisher noch nicht absteigend möglich

Peter Schulze added 7 commits April 24, 2024 11:42

history rework

437a842

cleanup (after rexstan use)

2cb0637

fixing bad fixes

9f87f95

(reverting const types, since only php 8.3+ | bringing back else area |

more cleanup ...

ee81d88

ran cs-fix

900e25c

cs fix

2fd941a

rex stan: next round ...

a3238f3

warning and less table columns if reference dataset_id is missing or …

b8f423d

…does not result in an existing dataset rex stan: more fixes

dergel reviewed May 20, 2024

View reviewed changes

Merge branch 'yakamara:master' into bitshifters-history-rework

28118db

Peter Schulze added 2 commits June 11, 2024 17:04

satisfying cs fixer

6737610

dergel reviewed Jun 20, 2024

View reviewed changes

Sortierung der Historyliste - neueste zuerst

01570ea

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

history rework #1502

history rework #1502

bitshiftersgmbh commented Apr 24, 2024 •

edited

Loading

skerbis commented Apr 26, 2024

bitshiftersgmbh commented Apr 26, 2024

bitshiftersgmbh commented Apr 28, 2024

skerbis commented Apr 29, 2024

dergel May 20, 2024

bitshiftersgmbh Jun 7, 2024

dergel May 20, 2024

bitshiftersgmbh Jun 7, 2024

dergel commented May 20, 2024

bitshiftersgmbh commented May 20, 2024

bitshiftersgmbh commented Jun 7, 2024

bitshiftersgmbh commented Jun 7, 2024

dergel Jun 20, 2024

dergel Jun 20, 2024

dergel commented Jun 20, 2024


		$list = rex_list::factory($listQuery, defaultSort: [
		'timestamp' => 'asc',

history rework #1502

Are you sure you want to change the base?

history rework #1502

Conversation

bitshiftersgmbh commented Apr 24, 2024 • edited Loading

skerbis commented Apr 26, 2024

bitshiftersgmbh commented Apr 26, 2024

bitshiftersgmbh commented Apr 28, 2024

skerbis commented Apr 29, 2024

dergel May 20, 2024

Choose a reason for hiding this comment

bitshiftersgmbh Jun 7, 2024

Choose a reason for hiding this comment

dergel May 20, 2024

Choose a reason for hiding this comment

bitshiftersgmbh Jun 7, 2024

Choose a reason for hiding this comment

dergel commented May 20, 2024

bitshiftersgmbh commented May 20, 2024

bitshiftersgmbh commented Jun 7, 2024

bitshiftersgmbh commented Jun 7, 2024

dergel Jun 20, 2024

Choose a reason for hiding this comment

dergel Jun 20, 2024

Choose a reason for hiding this comment

dergel commented Jun 20, 2024

bitshiftersgmbh commented Apr 24, 2024 •

edited

Loading