mirror of
				https://github.com/invoiceninja/invoiceninja.git
				synced 2025-11-04 03:29:03 -05:00 
			
		
		
		
	Merge pull request #4792 from beganovich/v5-2801-fixes-for-tax-columns-hiding
(v5) PDF: Fix bug with hiding non-present multiple taxes
This commit is contained in:
		
						commit
						823e54c968
					
				@ -314,6 +314,12 @@ class Design extends BaseDesign
 | 
				
			|||||||
                $elements[] = ['element' => 'th', 'content' => $column . '_label', 'properties' => ['data-ref' => "{$type}_table-" . substr($column, 1) . '-th', 'style' => 'display: none;']];
 | 
					                $elements[] = ['element' => 'th', 'content' => $column . '_label', 'properties' => ['data-ref' => "{$type}_table-" . substr($column, 1) . '-th', 'style' => 'display: none;']];
 | 
				
			||||||
            } elseif ($column == '$product.quantity' && !$this->client->company->enable_product_quantity) {
 | 
					            } elseif ($column == '$product.quantity' && !$this->client->company->enable_product_quantity) {
 | 
				
			||||||
                $elements[] = ['element' => 'th', 'content' => $column . '_label', 'properties' => ['data-ref' => "{$type}_table-" . substr($column, 1) . '-th', 'style' => 'display: none;']];
 | 
					                $elements[] = ['element' => 'th', 'content' => $column . '_label', 'properties' => ['data-ref' => "{$type}_table-" . substr($column, 1) . '-th', 'style' => 'display: none;']];
 | 
				
			||||||
 | 
					            } elseif ($column == '$product.tax_rate1') {
 | 
				
			||||||
 | 
					                $elements[] = ['element' => 'th', 'content' => $column . '_label', 'properties' => ['data-ref' => "{$type}_table-product.tax1-th", 'hidden' => $this->client->getSetting('hide_empty_columns_on_pdf')]];
 | 
				
			||||||
 | 
					            } elseif ($column == '$product.tax_rate2') {
 | 
				
			||||||
 | 
					                $elements[] = ['element' => 'th', 'content' => $column . '_label', 'properties' => ['data-ref' => "{$type}_table-product.tax2-th", 'hidden' => $this->client->getSetting('hide_empty_columns_on_pdf')]];
 | 
				
			||||||
 | 
					            } elseif ($column == '$product.tax_rate3') {
 | 
				
			||||||
 | 
					                $elements[] = ['element' => 'th', 'content' => $column . '_label', 'properties' => ['data-ref' => "{$type}_table-product.tax3-th", 'hidden' => $this->client->getSetting('hide_empty_columns_on_pdf')]];
 | 
				
			||||||
            } else {
 | 
					            } else {
 | 
				
			||||||
                $elements[] = ['element' => 'th', 'content' => $column . '_label', 'properties' => ['data-ref' => "{$type}_table-" . substr($column, 1) . '-th', 'hidden' => $this->client->getSetting('hide_empty_columns_on_pdf')]];
 | 
					                $elements[] = ['element' => 'th', 'content' => $column . '_label', 'properties' => ['data-ref' => "{$type}_table-" . substr($column, 1) . '-th', 'hidden' => $this->client->getSetting('hide_empty_columns_on_pdf')]];
 | 
				
			||||||
            }
 | 
					            }
 | 
				
			||||||
@ -337,6 +343,7 @@ class Design extends BaseDesign
 | 
				
			|||||||
        if (count($items) == 0) {
 | 
					        if (count($items) == 0) {
 | 
				
			||||||
            return [];
 | 
					            return [];
 | 
				
			||||||
        }
 | 
					        }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
        if ($type == 'delivery_note') {
 | 
					        if ($type == 'delivery_note') {
 | 
				
			||||||
            foreach ($items as $row) {
 | 
					            foreach ($items as $row) {
 | 
				
			||||||
                $element = ['element' => 'tr', 'elements' => []];
 | 
					                $element = ['element' => 'tr', 'elements' => []];
 | 
				
			||||||
@ -393,6 +400,12 @@ class Design extends BaseDesign
 | 
				
			|||||||
                        $element['elements'][] = ['element' => 'td', 'content' => $row['$product.quantity'], 'properties' => ['data-ref' => 'product_table-product.quantity-td', 'style' => 'display: none;']];
 | 
					                        $element['elements'][] = ['element' => 'td', 'content' => $row['$product.quantity'], 'properties' => ['data-ref' => 'product_table-product.quantity-td', 'style' => 'display: none;']];
 | 
				
			||||||
                    } elseif ($cell == '$task.hours') {
 | 
					                    } elseif ($cell == '$task.hours') {
 | 
				
			||||||
                        $element['elements'][] = ['element' => 'td', 'content' => $row['$task.quantity'], 'properties' => ['data-ref' => 'task_table-task.hours-td']];
 | 
					                        $element['elements'][] = ['element' => 'td', 'content' => $row['$task.quantity'], 'properties' => ['data-ref' => 'task_table-task.hours-td']];
 | 
				
			||||||
 | 
					                    } elseif ($cell == '$product.tax_rate1') {
 | 
				
			||||||
 | 
					                        $element['elements'][] = ['element' => 'td', 'content' => $row[$cell], 'properties' => ['data-ref' => 'product_table-product.tax1-td']];
 | 
				
			||||||
 | 
					                    } elseif ($cell == '$product.tax_rate2') {
 | 
				
			||||||
 | 
					                        $element['elements'][] = ['element' => 'td', 'content' => $row[$cell], 'properties' => ['data-ref' => 'product_table-product.tax2-td']];
 | 
				
			||||||
 | 
					                    } elseif ($cell == '$product.tax_rate3') {
 | 
				
			||||||
 | 
					                        $element['elements'][] = ['element' => 'td', 'content' => $row[$cell], 'properties' => ['data-ref' => 'product_table-product.tax3-td']];
 | 
				
			||||||
                    } else {
 | 
					                    } else {
 | 
				
			||||||
                        $element['elements'][] = ['element' => 'td', 'content' => $row[$cell], 'properties' => ['data-ref' => "{$_type}_table-" . substr($cell, 1) . '-td']];
 | 
					                        $element['elements'][] = ['element' => 'td', 'content' => $row[$cell], 'properties' => ['data-ref' => "{$_type}_table-" . substr($cell, 1) . '-td']];
 | 
				
			||||||
                    }
 | 
					                    }
 | 
				
			||||||
 | 
				
			|||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user