From 08490d647c2ae17aef6b72001e28cd48903edb78 Mon Sep 17 00:00:00 2001
From: frytimo <tim@fusionpbx.com>
Date: Fri, 23 Jan 2026 12:41:42 -0400
Subject: [PATCH] Feature codes add cell padding to PDF output (#7716)

---
 app/feature_codes/feature_codes.php | 30 ++++++++++++++++++++++-------
 1 file changed, 23 insertions(+), 7 deletions(-)

diff --git a/app/feature_codes/feature_codes.php b/app/feature_codes/feature_codes.php
index 7d7ef61db4..41e3e46d93 100644
--- a/app/feature_codes/feature_codes.php
+++ b/app/feature_codes/feature_codes.php
@@ -81,10 +81,14 @@
 
 		//create pdf
 		$pdf = new FPDF('P', 'mm', 'A4');
-		$pdf->SetAutoPageBreak(true, 15);
+		$pdf->SetAutoPageBreak(false); //handle page breaks manually
 		$pdf->AddPage();
 		$pdf->SetFont('Arial', 'B', 16);
 
+		//page dimensions
+		$page_height = 297; //A4 height in mm
+		$bottom_margin = 15;
+
 		//title
 		$pdf->Cell(0, 10, $text['title-feature_codes'], 0, 1, 'C');
 		$pdf->Ln(5);
@@ -141,7 +145,14 @@
 							}
 						}
 					}
-					$row_height = max($line_height, $num_lines * $line_height);
+					$cell_padding = 2; //1mm top + 1mm bottom
+					$row_height = max($line_height, $num_lines * $line_height) + $cell_padding;
+
+					//check if row fits on current page, if not add new page
+					if ($pdf->GetY() + $row_height > $page_height - $bottom_margin) {
+						$pdf->AddPage();
+						$pdf->SetFont('Arial', '', 9);
+					}
 
 					//save starting position
 					$x = $pdf->GetX();
@@ -166,7 +177,7 @@
 					//draw description cell border and use MultiCell for text
 					$desc_x = $x + $col_widths[0] + $col_widths[1];
 					$pdf->Rect($desc_x, $y, $col_widths[2], $row_height);
-					$pdf->SetXY($desc_x + 1, $y + 1);
+					$pdf->SetXY($desc_x + 1, $y + ($cell_padding / 2));
 					$pdf->MultiCell($col_widths[2] - 2, $line_height, $feature_description, 0, 'L');
 
 					//draw border and vertically centered text for raw column
@@ -199,7 +210,14 @@
 							}
 						}
 					}
-					$row_height = max($line_height, $num_lines * $line_height);
+					$cell_padding = 2; //1mm top + 1mm bottom
+					$row_height = max($line_height, $num_lines * $line_height) + $cell_padding;
+
+					//check if row fits on current page, if not add new page
+					if ($pdf->GetY() + $row_height > $page_height - $bottom_margin) {
+						$pdf->AddPage();
+						$pdf->SetFont('Arial', '', 9);
+					}
 
 					//save starting position
 					$x = $pdf->GetX();
@@ -224,7 +242,7 @@
 					//draw description cell border and use MultiCell for text
 					$desc_x = $x + $col_widths[0] + $col_widths[1];
 					$pdf->Rect($desc_x, $y, $col_widths[2], $row_height);
-					$pdf->SetXY($desc_x + 1, $y + 1);
+					$pdf->SetXY($desc_x + 1, $y + ($cell_padding / 2));
 					$pdf->MultiCell($col_widths[2] - 2, $line_height, $feature_description, 0, 'L');
 
 					//move to next row
@@ -316,5 +334,3 @@
 
 //include footer
 	require_once "resources/footer.php";
-
-?>