yform icon indicating copy to clipboard operation
yform copied to clipboard

history rework

Open bitshiftersgmbh opened this issue 1 year ago • 9 comments

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

bitshiftersgmbh avatar Apr 24 '24 09:04 bitshiftersgmbh

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

skerbis avatar Apr 26 '24 14:04 skerbis

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

bitshiftersgmbh avatar Apr 26 '24 15:04 bitshiftersgmbh

@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 ...

bitshiftersgmbh avatar Apr 28 '24 15:04 bitshiftersgmbh

Super. 👍🏻

skerbis avatar Apr 29 '24 06:04 skerbis

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.

dergel avatar May 20 '24 20:05 dergel

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

bitshiftersgmbh avatar May 20 '24 20:05 bitshiftersgmbh

@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 avatar Jun 07 '24 15:06 bitshiftersgmbh

@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).

bitshiftersgmbh avatar Jun 07 '24 15:06 bitshiftersgmbh

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

dergel avatar Jun 20 '24 10:06 dergel

Bist du hier noch dran?

dergel avatar May 02 '25 11:05 dergel

Bist du hier noch dran?

Ja bin ich. Hab's auf dem Schirm und brauche es auch für ein aktuelles Projekt dringend korrekt. Kommt in den nächsten 1,5 Monaten!

bitshiftersgmbh avatar May 11 '25 12:05 bitshiftersgmbh

piep piep piep :)

dergel avatar Aug 27 '25 11:08 dergel