projekt kivitendo

Fehler tritt nach bisect das erste mal bei 55a2f137cfb3 auf

Ich bin jetzt auch darüber gestolpert und finde damit ist das Pflichtenheft aktuell leider sehr unkomfortabel.

Ich hab hier einen aktuelle ckeditor 4.7.3 runtergeladen und drüberkopiert, damit wurde das Verhalten aber nicht besser:

sudo wget https://download.cksource.com/CKEditor/CKEditor/CKEditor%204.7.3/ckeditor_4.7.3_basic.zip
unzip ckeditor_4.7.3_basic.zip
sudo rsync --delete -av ckeditor/ /var/www/kivitendo-erp/js/ckeditor/
sudo service apache2 restart

Ich kann mir aber auch nicht vorstellen, dass so etwas fundamentales im ckeditor in der Standardversion nicht funktioniert.

Hat jmd vielleicht eine Idee hierzu oder was brauchst vielleicht an finanziellen Mittel.
Soweit kann man sich ja behelfen wenn man die <space> in Zusammenhang mit der <shift> Taste benutzt.
Aber das ist schön ziemlich blöd dieser bug und ich hab keine Ahnung wie ich das debuggen oder gar beheben kann.

Ich hab ein wenig rumprobiert. Es scheint sich um eine Inkompatibilität mit dem Tree-Modul zu handeln.

Weiterhin habe ich dann versucht, das Tree-Modul mal zu aktualisieren. Das Problem: die aktuelle Version ist viel neuer, anscheinend nahezu ein Rewrite, und der existierende Code passt vorn und hinten nicht mehr dazu, wie die neue Version zu nutzen ist.

Aber immerhin: mit einem aktualisierten jstree-Code funktioniert plötzlich die Leertaste im Editor wieder. Der Tree funktioniert dafür halt nicht.

Problem ist nun gefixt. Ein Update des jsTree-Scripts schließe ich schlicht aus — das erfordert praktisch die Anpassung all unseres Codes, der was mit Bäumen macht (ist nur das Pflichtenheft, aber das ist massiv viel Code).

Statt dessen scheint das Problem daher zu stammen, dass jsTree immer gewisse Key-Event-Handler installiert (Cursor-Tasten, Space, Escape), auch wenn das jsTree-Plugin "hotkeys" nicht benutzt wird. Diese Handler scheinen Priorität vor den Handlern zu haben, die der ckEditor zusammen mit dem inline_resize-Plugin installiert.

Den Code einfach entfernt, und siehe da, die Leertaste kommt wieder richtig im ckEditor an.