data/mahnung
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(mahnung): Button via direktem print statt resprints — sonst kein Output [deploy]
All checks were successful
Deploy mahnung / deploy (push) Successful in 13s
Details

Browse source 
REGRESSION aus letztem Commit: $this->resprints wurde nicht gerendert.

addMoreActionsButtons-Hook in compta/facture/card.php wertet zwar $reshook aus
(empty → Default-Buttons), printet aber $hookmanager->resPrint an dieser Stelle
nicht — anders als bei formObjectOptions. Der Hook muss seinen HTML-Output
direkt selbst per print ausgeben.

Direkter print ist hier sicher: der Hook wird innerhalb des dafür vorgesehenen
<div class="tabsAction"> aufgerufen, kein HTML-Layoutbruch möglich.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
Eduard Wisch 2026-05-14 10:33:17 +02:00
parent e7375ac559
commit 7031bea118
1 changed files with 6 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
class/actions_mahnung.class.php
View file

@ -83,9 +83,12 @@ class ActionsMahnung
// Import Zeilen / Löschen / Auf anderen Kunden übertragen.
$spacer = '<div class="mahnung-row-break" style="flex-basis:100%;height:0;width:0;"></div>';
// Über $this->resprints zurückgeben statt direktem print — sonst kann der Output
// mitten in HTML-Fragmenten landen. Dolibarr printet danach $hookmanager->resPrint.
$this->resprints = $spacer.$btn;
// WICHTIG: addMoreActionsButtons braucht direkten print, nicht $this->resprints.
// card.php wertet zwar $reshook aus, druckt aber $hookmanager->resPrint an dieser
// Stelle NICHT — anders als bei formObjectOptions. Output direkt in die tabsAction-Div
// ist hier sicher, weil der Hook bereits innerhalb des dafür vorgesehenen
// Container-Divs aufgerufen wird.
print $spacer.$btn;
return 0;
}