From de0b0ee97c3d1a3395c52f9c124b5812552a3e4d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tomasz=20Chili=C5=84ski?= <tomasz.chilinski@chilan.com>
Date: Thu, 1 Jun 2023 12:42:49 +0200
Subject: [PATCH 1/7] improvement: show cash source name in imported operations
 pending for confirmation

---
 modules/cashimport.php                 | 17 ++++++++++++-----
 templates/default/cash/cashimport.html | 11 +++++++++--
 2 files changed, 21 insertions(+), 7 deletions(-)

diff --git a/modules/cashimport.php b/modules/cashimport.php
index f0c9239871..198d547bb4 100644
--- a/modules/cashimport.php
+++ b/modules/cashimport.php
@@ -257,11 +257,18 @@
 
 $divisions[0] = array('id' => 0, 'name' => '');
 
-if ($importlist = $DB->GetAll('SELECT i.*, c.divisionid
-	FROM cashimport i
-	LEFT JOIN customerview c ON (i.customerid = c.id)
-	WHERE i.closed = 0 AND i.value > 0
-	ORDER BY i.id')) {
+if ($importlist = $DB->GetAll(
+    'SELECT
+        i.*,
+        c.divisionid,
+        cs.name AS sourcename
+    FROM cashimport i
+    LEFT JOIN customerview c ON i.customerid = c.id
+    LEFT JOIN cashsources cs ON cs.id = i.sourceid
+    WHERE i.closed = 0
+        AND i.value > 0
+    ORDER BY i.id'
+)) {
     $listdata['total'] = count($importlist);
 
     foreach ($importlist as $idx => $row) {
diff --git a/templates/default/cash/cashimport.html b/templates/default/cash/cashimport.html
index e758e7a445..97ca163c16 100644
--- a/templates/default/cash/cashimport.html
+++ b/templates/default/cash/cashimport.html
@@ -35,6 +35,7 @@ <H1>{$layout.pagetitle}</H1>
 		<col style="width: 1%;">
 		<col style="width: 1%;">
 		<col style="width: 1%;">
+		<col style="width: 1%;">
 		<col style="width: 25%;">
 		<col style="width: 42%;">
 		<col style="width: 32%;">
@@ -45,6 +46,9 @@ <H1>{$layout.pagetitle}</H1>
 			<td>
 				{trans("ID:")}
 			</td>
+			<td>
+				{trans("Source")}
+			</td>
 			<td>
 				{trans("Date:")}
 			</td>
@@ -69,7 +73,7 @@ <H1>{$layout.pagetitle}</H1>
 		{foreach $divisions as $division}
 			{if $division.list}
 				<tr>
-					<td colspan="4" class="dark">
+					<td colspan="5" class="dark">
 						{icon name="division"}
 						<strong>{$division.name|escape}</strong>
 					</td>
@@ -82,6 +86,9 @@ <H1>{$layout.pagetitle}</H1>
 					<td>
 						{$item.id|string_format:"%04d"}
 					</td>
+					<td>
+						<strong>{$item.sourcename|escape}</strong>
+					</td>
 					<td class="nobr">
 						{$item.date|date_format:"Y-m-d"}
 					</td>
@@ -113,7 +120,7 @@ <H1>{$layout.pagetitle}</H1>
 		{/foreach}
 		{if !$listdata.total}
 			<tr>
-				<td class="empty-table" colspan="7">
+				<td class="empty-table" colspan="8">
 					{trans("No cash operations to import.")}
 				</td>
 			</tr>

From 5952622a408b630dfc225e105f336a2d2cfeb8ff Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tomasz=20Chili=C5=84ski?= <tomasz.chilinski@chilan.com>
Date: Thu, 1 Jun 2023 12:43:28 +0200
Subject: [PATCH 2/7] improvement: show cash source name in imported operations
 pending for confirmation (changelog entry)

---
 doc/ChangeLog | 1 +
 1 file changed, 1 insertion(+)

diff --git a/doc/ChangeLog b/doc/ChangeLog
index 48b7622bab..a3822de70f 100644
--- a/doc/ChangeLog
+++ b/doc/ChangeLog
@@ -509,6 +509,7 @@ version 28-git (????-??-??):
   - enhancement: 'without start date' assignment filter added in customer search form [chilan]
   - improvement: show customer id for addresses in 'merge' mode lms-teryt.php script [chilan]
   - enhancement: added division property to network devices [chilan]
+  - improvement: show source name in imported operation list pending for confirmation [chilan]
 
 version 27.0 (2021-08-20):
 

From 6ebde1d0af1a296220e206732f3b58dfc6f4b3fe Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tomasz=20Chili=C5=84ski?= <tomasz.chilinski@chilan.com>
Date: Thu, 1 Jun 2023 12:46:24 +0200
Subject: [PATCH 3/7] improvement: show cash source name in imported operations
 pending for confirmation (small visual fix)

---
 templates/default/cash/cashimport.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/templates/default/cash/cashimport.html b/templates/default/cash/cashimport.html
index 97ca163c16..8c43a1a09e 100644
--- a/templates/default/cash/cashimport.html
+++ b/templates/default/cash/cashimport.html
@@ -128,7 +128,7 @@ <H1>{$layout.pagetitle}</H1>
 	</tbody>
 	<tfoot>
 		<tr class="lms-ui-button-panel">
-			<td class="nobr" colspan="3">
+			<td class="nobr" colspan="4">
 				<label>
 					<input type="checkbox" name="allbox" class="lms-ui-multi-check-all" value="1" form="cashimport">
 					{trans("Check All")}

From 556358b00024f8d2d613366cc6d49a4ee2b40534 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tomasz=20Chili=C5=84ski?= <tomasz.chilinski@chilan.com>
Date: Fri, 2 Jun 2023 20:13:15 +0200
Subject: [PATCH 4/7] improvement/devel: 'get_customer_sms_options' plugin hook
 introduced which allows to collect specific sms-like section configuration
 variable names

---
 lib/LMS.class.php                      | 2 +-
 lib/LMSManagers/LMSCustomerManager.php | 7 +++++++
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/lib/LMS.class.php b/lib/LMS.class.php
index c2f477b404..d9977ddd22 100644
--- a/lib/LMS.class.php
+++ b/lib/LMS.class.php
@@ -3286,7 +3286,7 @@ public function SendSMS($number, $message, $messageid = null, $sms_options = nul
 
         // message ID must be unique
         if (!$messageid) {
-            $messageid = '0.' . time();
+            $messageid = '0.' . microtime(true);
         }
 
         $message = preg_replace("/\r/", "", $message);
diff --git a/lib/LMSManagers/LMSCustomerManager.php b/lib/LMSManagers/LMSCustomerManager.php
index 473fe6b870..4e43b1e9b1 100644
--- a/lib/LMSManagers/LMSCustomerManager.php
+++ b/lib/LMSManagers/LMSCustomerManager.php
@@ -2717,6 +2717,8 @@ public function isTelecomServiceSuggested($customerid)
 
     public function getCustomerSMSOptions()
     {
+        global $LMS;
+
         $options = array();
 
         $variable_mapping = array(
@@ -2737,6 +2739,11 @@ public function getCustomerSMSOptions()
             'message_template' => 'sms-customers.message_template',
         );
 
+        $variable_mapping = $LMS->executeHook(
+            'get_customer_sms_options',
+            $variable_mapping
+        );
+
         foreach ($variable_mapping as $option_name => $variable_name) {
             if (is_array($variable_name)) {
                 $exists = false;

From 57cff1ce8e5527216988f17263817cc107100fab Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tomasz=20Chili=C5=84ski?= <tomasz.chilinski@chilan.com>
Date: Fri, 2 Jun 2023 20:14:03 +0200
Subject: [PATCH 5/7] improvement/devel: 'get_customer_sms_options' plugin hook
 introduced which allows to collect specific sms-like section configuration
 variable names (changelog entry)

---
 doc/ChangeLog | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/doc/ChangeLog b/doc/ChangeLog
index a3822de70f..76e6af5a21 100644
--- a/doc/ChangeLog
+++ b/doc/ChangeLog
@@ -510,6 +510,8 @@ version 28-git (????-??-??):
   - improvement: show customer id for addresses in 'merge' mode lms-teryt.php script [chilan]
   - enhancement: added division property to network devices [chilan]
   - improvement: show source name in imported operation list pending for confirmation [chilan]
+  - improvement/devel: 'get_customer_sms_options' plugin hook introduced which allows to collect
+    specific sms-like section configuration variable names [chilan]
 
 version 27.0 (2021-08-20):
 

From edf98a4e09aa1c3fd6ba4893189cc5e1c727a756 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tomasz=20Chili=C5=84ski?= <tomasz.chilinski@chilan.com>
Date: Mon, 5 Jun 2023 12:48:08 +0200
Subject: [PATCH 6/7] improvement: send document confirm notifications if
 document is created with 'closed' status

---
 modules/documentadd.php | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/modules/documentadd.php b/modules/documentadd.php
index 9655bbdca5..e7422a9262 100644
--- a/modules/documentadd.php
+++ b/modules/documentadd.php
@@ -485,12 +485,16 @@
                 )
             );
             $document = $hook_data['document'];
-        }
 
-        if ($LMS->DocumentExists($docid) && !empty($document['confirmdate'])) {
-            $document['id'] = $docid;
-            $document['fullnumber'] = $fullnumber;
-            $LMS->NewDocumentCustomerNotifications($document);
+            if (isset($document['closed'])) {
+                $LMS->CommitDocuments(array($docid), false, false);
+            }
+
+            if (!empty($document['confirmdate'])) {
+                $document['id'] = $docid;
+                $document['fullnumber'] = $fullnumber;
+                $LMS->NewDocumentCustomerNotifications($document);
+            }
         }
 
         if (!isset($document['reuse'])) {

From 102f1078ca2feb20afe482bfdeb5b0961744e4f4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tomasz=20Chili=C5=84ski?= <tomasz.chilinski@chilan.com>
Date: Mon, 5 Jun 2023 12:48:37 +0200
Subject: [PATCH 7/7] improvement: send document confirm notifications if
 document is created with 'closed' status (changelog entry)

---
 doc/ChangeLog | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/doc/ChangeLog b/doc/ChangeLog
index 76e6af5a21..e725e478af 100644
--- a/doc/ChangeLog
+++ b/doc/ChangeLog
@@ -512,6 +512,8 @@ version 28-git (????-??-??):
   - improvement: show source name in imported operation list pending for confirmation [chilan]
   - improvement/devel: 'get_customer_sms_options' plugin hook introduced which allows to collect
     specific sms-like section configuration variable names [chilan]
+  - improvement: send document confirm notifications if document is created with 'closed'
+    status [chilan]
 
 version 27.0 (2021-08-20):