From c18d75719bfd0fc8064538b13e87fc477ce3962a Mon Sep 17 00:00:00 2001
From: sparkyx <danydb@noalyss.eu>
Date: Fri, 27 Feb 2026 10:03:10 +0100
Subject: [PATCH] Fix Bug chrome : cannot save operation details

---
 html/js/acc_ledger.js                   | 18 ++++++++----------
 include/ajax/ajax_view_action.php       | 14 +++++++++++++-
 include/template/ledger_detail_ach.php  |  2 +-
 include/template/ledger_detail_fin.php  |  2 +-
 include/template/ledger_detail_misc.php |  2 +-
 include/template/ledger_detail_ven.php  |  6 +++---
 6 files changed, 27 insertions(+), 17 deletions(-)

diff --git a/html/js/acc_ledger.js b/html/js/acc_ledger.js
index fb7736fcd..5c02037c3 100644
--- a/html/js/acc_ledger.js
+++ b/html/js/acc_ledger.js
@@ -1297,14 +1297,12 @@ function search_letter(obj) {
  */
 function op_save(obj) {
     try {
-        var queryString = id$(obj).serialize(true);
-        queryString ["gDossier"] = obj.gDossier.value;
-        var rapt2 = "rapt" + obj.whatdiv.value;
-        queryString ["rapt"] = id$(rapt2).value;
-        queryString  ["jr_id"] = obj.jr_id.value;
-        var jr_id = obj.jr_id.value;
-        queryString ["div"] = obj.whatdiv.value;
-        var divid = obj.whatdiv.value;
+        var queryString = Form.serialize(obj,true);
+        var divid = queryString['whatdiv'];
+        queryString["div"]  = queryString['whatdiv'];
+        queryString["rapt"] = id$("rapt"+divid).value;
+        var jr_id = queryString['jr_id'];
+        
         queryString ["act"] = "save";
         queryString ["op"] = "ledger";
         queryString ["jr_note"]=encodeURI(tinyMCE.get("jrn_note"+divid).getContent());
@@ -1346,7 +1344,7 @@ function op_save(obj) {
                         }
                         new Ajax.Request('ajax_misc.php', {
                             parameters: {
-                                'gDossier': obj.gDossier.value,
+                                'gDossier': queryString ["gDossier"],
                                 'act': 'de',
                                 'op': 'ledger',
                                 'jr_id': jr_id,
@@ -1359,7 +1357,7 @@ function op_save(obj) {
                                     id$(divid).innerHTML = unescape(getNodeText(html[0]));
                                     id$(divid).innerHTML.evalScripts();
                                     remove_waiting_box();
-                                    noalyss.refresh_note(jr_id,obj.gDossier.value);
+                                    noalyss.refresh_note(jr_id,queryString ["gDossier"]);
                                 } catch (e) {
                                     console.error("D1. op_save")
                                     alert_box("1038" + e.message)
diff --git a/include/ajax/ajax_view_action.php b/include/ajax/ajax_view_action.php
index 2af8c37d5..a72e3ac2a 100644
--- a/include/ajax/ajax_view_action.php
+++ b/include/ajax/ajax_view_action.php
@@ -42,7 +42,15 @@ if ( ($g_user->can_write_action($ag_id) == true || $g_user->can_read_action($ag_
         $action=  NOALYSS_URL."/do.php?".http_build_query(array("gDossier"=>Dossier::id(),"ag_id"=>$ag_id,"ac"=>$menu->get('code_follow'),"sa"=>"detail"));
         $code='ok';
 ?>
+<ul class="aligned-block">
+    <li>
+        
 <a href="<?php echo $action?>" target="_blank" class="smallbutton"><?php echo _("Modifier")?> </a>
+    </li>
+    <li>
+        <?php echo HtmlInput::button_close($div)?>
+    </li>
+</ul>
 <?php
 }
 else
@@ -52,9 +60,13 @@ else
 	<div style="margin:0px;padding:0px;background-color:red;text-align:center;">
         <h2 class="error"><?php echo $forbidden ?></h2>;
 </div>
+<ul class="aligned-block">
+    <li>
+        <?php echo HtmlInput::button_close($div)?>
+    </li>
+</ul>
 	<?php 
 }
-echo HtmlInput::button_close($div);
 $response =  ob_get_clean();
 $html=escape_xml($response);
 header('Content-type: text/xml; charset=UTF-8');
diff --git a/include/template/ledger_detail_ach.php b/include/template/ledger_detail_ach.php
index 7efab4a3c..698f26ed6 100644
--- a/include/template/ledger_detail_ach.php
+++ b/include/template/ledger_detail_ach.php
@@ -23,7 +23,7 @@ $dossier_id=Dossier::id();
     ?>
 
     <?php if ($access == 'W') : ?>
-        <form  class="print" onsubmit="return op_save(this);">
+        <form  class="print" id="<?=$div?>_frm" onsubmit="return op_save('<?=$div?>_frm');">
         <?php endif; ?>
 
         <?php echo HtmlInput::hidden('whatdiv', $div) . HtmlInput::hidden('jr_id', $jr_id) . dossier::hidden(); ?>
diff --git a/include/template/ledger_detail_fin.php b/include/template/ledger_detail_fin.php
index 948e238c6..a9f867e21 100644
--- a/include/template/ledger_detail_fin.php
+++ b/include/template/ledger_detail_fin.php
@@ -18,7 +18,7 @@ $dossier_id=Dossier::id();
 
 ?>
     <?php if ( $access=='W') : ?>
-<form class="print" onsubmit="return op_save(this);">
+  <form  class="print" id="<?=$div?>_frm" onsubmit="return op_save('<?=$div?>_frm');">
    <?php endif; ?>
 
     <?php echo HtmlInput::hidden('whatdiv',$div).HtmlInput::hidden('jr_id',$jr_id).dossier::hidden();?>
diff --git a/include/template/ledger_detail_misc.php b/include/template/ledger_detail_misc.php
index 2da28cd78..b160ce7ca 100644
--- a/include/template/ledger_detail_misc.php
+++ b/include/template/ledger_detail_misc.php
@@ -22,7 +22,7 @@ $dossier_id=Dossier::id();
 <div class="content">
 
     <?php if ( $access=='W') : ?>
-<form class="print" onsubmit="return op_save(this);">
+  <form  class="print" id="<?=$div?>_frm" onsubmit="return op_save('<?=$div?>_frm');">
    <?php endif; ?>
 
     <?php echo HtmlInput::hidden('whatdiv',$div).HtmlInput::hidden('jr_id',$jr_id).dossier::hidden();?>
diff --git a/include/template/ledger_detail_ven.php b/include/template/ledger_detail_ven.php
index 0c04d3655..2ba392279 100644
--- a/include/template/ledger_detail_ven.php
+++ b/include/template/ledger_detail_ven.php
@@ -25,9 +25,9 @@ $str_anc = "";
     $owner = new Noalyss_Parameter_Folder($cn);
     ?>
 
-    <?php if ($access == 'W') :
-        echo '<form class="print" onsubmit="return op_save(this);">';
-    endif; ?>
+    <?php if ($access == 'W') :?>
+          <form  class="print" id="<?=$div?>_frm" onsubmit="return op_save('<?=$div?>_frm');">
+    <?php endif; ?>
 
     <?php echo HtmlInput::hidden('whatdiv', $div) . HtmlInput::hidden('jr_id', $jr_id) . dossier::hidden(); ?>
     <table style="width:100%">