Skip to content

Color3 was null #2520

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
JanKallman merged 2 commits into from
Sep 14, 2026
+174 −2
Merged

Color3 was null #2520

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
e8d73b2
Added tests for LookupColor/PDF issue
swmal Sep 14, 2026
88c343b
Fixed color3
AdrianEPPlus Sep 14, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
155 changes: 155 additions & 0 deletions src/EPPlus.Export.Pdf.Tests/Helpers/ColorIssueTestHelper.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,155 @@
using OfficeOpenXml;
using OfficeOpenXml.Style;
using OfficeOpenXml.Table;
using System;
using System.Collections.Generic;
using System.Data;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace EPPlus.Export.Pdf.Tests.Helpers
{
internal class ColorIssueTestHelper
{
public TableStyles TableStyle { get; set; } = TableStyles.Dark3;

public ExcelPackage CreateWorkbook()
{
InitDataTable();
var package = new ExcelPackage();
var sheet = package.Workbook.Worksheets.Add("Html export sample 2");
var tableRange = sheet.Cells["A1"].LoadFromDataTable(_dataTable, true, TableStyle);

//Configure the table
var table = sheet.Tables.GetFromRange(tableRange);
table.Sort(x => x.SortBy.ColumnNamed("Population", eSortOrder.Descending));
table.ShowTotal = true;
table.Columns[0].TotalsRowLabel = "Total";
table.Columns[1].TotalsRowFunction = RowFunctions.Sum;
table.Columns[2].TotalsRowFunction = RowFunctions.Sum;

//Add column for population density
table.Columns.Add(1);
tableRange = table.Range;
table.Columns[3].CalculatedColumnFormula = $"{table.Name}[[#This Row],[Population]]/{table.Name}[[#This Row],[Area (km2)]]";
table.Columns[3].Name = "Density";
table.Columns[3].TotalsRowFunction = RowFunctions.Average;
sheet.Calculate();

//// format the header
table.Range.TakeColumnsBetween(1, 3).SkipRows(1).Style.HorizontalAlignment = ExcelHorizontalAlignment.Right;

// format the rows
var lastDataRow = tableRange.End.Row - 1;
sheet.Cells[tableRange.Start.Row, 1, lastDataRow, 1].Style.HorizontalAlignment = ExcelHorizontalAlignment.Left;
sheet.Cells[tableRange.Start.Row, 2, lastDataRow, 2].Style.Numberformat.Format = "#,##0";
sheet.Cells[tableRange.Start.Row, 3, lastDataRow, 3].Style.Numberformat.Format = "#,##0 \"km2\"";
sheet.Cells[tableRange.Start.Row, 4, lastDataRow, 4].Style.Numberformat.Format = "#,##0.0";

// format the total row
var totalRow = tableRange.End.Row;
sheet.Cells[totalRow, 1].Style.HorizontalAlignment = ExcelHorizontalAlignment.Left;
sheet.Cells[totalRow, 2].Style.Numberformat.Format = "#,##0";
sheet.Cells[totalRow, 3].Style.Numberformat.Format = "#,##0 \"km2\"";
sheet.Cells[totalRow, 4].Style.Numberformat.Format = "\"Avg: \"#,##0.0 ";
sheet.Cells.AutoFitColumns();

//Set the header and footer values
var text = sheet.HeaderFooter.OddHeader.Centered.AddText("EPPlus Sample 3");
text.FontSize = 18;
var webRootPath = "c:\\Temp";
var imageFile = Path.Combine(webRootPath, "img", "EPPlus-logo-small.png");
if (File.Exists(imageFile))
{
//sheet.HeaderFooter.OddHeader.LeftAligned.AddText("Logo:");
sheet.HeaderFooter.OddHeader.LeftAligned.AddImage(new FileInfo(imageFile));
}

sheet.HeaderFooter.OddFooter.Centered.AddPageNumber();
sheet.HeaderFooter.OddFooter.Centered.AddText(" of ");
sheet.HeaderFooter.OddFooter.Centered.AddNumberOfPages();

// Now copy the worksheet from the Allsvenskan2001 workbook
var file = "c:\\epplusTest\\Workbooks\\Allsvenskan2001.xlsx";
using var package2 = new ExcelPackage(file);
var aWs = package2.Workbook.Worksheets[0];
package.Workbook.Worksheets.Add(aWs.Name, aWs);
return package;
}

private DataTable _dataTable = null;


private void InitDataTable()
{
if (_dataTable != null) return;
_dataTable = new DataTable();
_dataTable.Columns.Add("Country", typeof(string));
_dataTable.Columns.Add("Population", typeof(int));
var areaCol = _dataTable.Columns.Add("Area", typeof(int));
areaCol.Caption = "Area (km2)";


_dataTable.Rows.Add("Sweden", 10409248, 450295);
_dataTable.Rows.Add("Norway", 5402171, 385178);
_dataTable.Rows.Add("Netherlands", 17553530, 41198);
_dataTable.Rows.Add("Finland", 5541806, 338145);
_dataTable.Rows.Add("Belgium", 11521238, 30510);
_dataTable.Rows.Add("Denmark", 5850189, 44493);
_dataTable.Rows.Add("Lithuania", 2801264, 65300);
_dataTable.Rows.Add("Greece", 10718565, 131940);
_dataTable.Rows.Add("Russia", 145734038, 3972400);
_dataTable.Rows.Add("Germany", 83124418, 357386);
_dataTable.Rows.Add("France", 64990511, 551695);
_dataTable.Rows.Add("Czech Republic", 10665677, 78866);
_dataTable.Rows.Add("Slovakia", 5459781, 49036);
_dataTable.Rows.Add("Spain", 47394223, 498468);
_dataTable.Rows.Add("Portugal", 10256193, 91568);
_dataTable.Rows.Add("United Kingdom", 67141684, 242495);
_dataTable.Rows.Add("Poland", 37921592, 312685);
_dataTable.Rows.Add("Albania", 2882740, 28748);
_dataTable.Rows.Add("Estonia", 1322920, 45339);
_dataTable.Rows.Add("Hungary", 9707499, 93030);
_dataTable.Rows.Add("Romania", 19186000, 238397);
_dataTable.Rows.Add("Italy", 60627291, 301338);
_dataTable.Rows.Add("Bulgaria", 7051608, 110994);
_dataTable.Rows.Add("Belarus", 9452617, 207600);
_dataTable.Rows.Add("Austria", 8891388, 83858);
_dataTable.Rows.Add("Switzerland", 8525611, 41290);
_dataTable.Rows.Add("Ireland", 4818690, 70273);
_dataTable.Rows.Add("Ukraine", 44246156, 603628);
_dataTable.Rows.Add("Iceland", 336713, 102775);
_dataTable.Rows.Add("Serbia", 6871547, 77453);
_dataTable.Rows.Add("Croatia", 4156405, 56594);
_dataTable.Rows.Add("Latvia", 1928459, 64589);
_dataTable.Rows.Add("Bosnia and Herzegovina", 3323925, 51129);
_dataTable.Rows.Add("Montenegro", 627809, 13812);
_dataTable.Rows.Add("Cyrprus", 1189265, 9251);
_dataTable.Rows.Add("Kosovo", 1798506, 10908);
_dataTable.Rows.Add("Slovenia", 2055496, 20273);
_dataTable.Rows.Add("Moldova", 4033963, 33846);
_dataTable.Rows.Add("North Macedonia", 2083374, 25713);
_dataTable.Rows.Add("United States", 331002651, 9833517);
_dataTable.Rows.Add("China", 1412600000, 9596961);
_dataTable.Rows.Add("India", 1417173173, 3287263);
_dataTable.Rows.Add("Japan", 125681593, 377975);
_dataTable.Rows.Add("Brazil", 214326223, 8515767);
_dataTable.Rows.Add("Canada", 38246108, 9984670);
_dataTable.Rows.Add("Australia", 25788215, 7692024);
_dataTable.Rows.Add("Mexico", 126014024, 1964375);
_dataTable.Rows.Add("South Africa", 59308690, 1221037);
_dataTable.Rows.Add("Egypt", 104258327, 1001450);
_dataTable.Rows.Add("Nigeria", 218541212, 923768);
_dataTable.Rows.Add("Argentina", 45808747, 2780400);
_dataTable.Rows.Add("Indonesia", 273523615, 1904569);
_dataTable.Rows.Add("South Korea", 51780579, 100210);
_dataTable.Rows.Add("Turkey", 84339067, 783562);
_dataTable.Rows.Add("Saudi Arabia", 34813871, 2149690);
_dataTable.Rows.Add("Israel", 9053300, 20770);
_dataTable.Rows.Add("New Zealand", 5084300, 268838);
_dataTable.Rows.Add("Thailand", 69950850, 513120);
_dataTable.Rows.Add("Vietnam", 98168833, 331212);
}
}
}
20 changes: 18 additions & 2 deletions src/EPPlus.Export.Pdf.Tests/PdfTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,13 +13,14 @@ Date Author Change
using EPPlus.Export.Pdf.Settings;
using EPPlus.Export.Pdf.Settings.PdfPageSizes;
using EPPlus.Export.Pdf.Tests;
using EPPlus.Export.Pdf.Tests.Helpers;
using OfficeOpenXml;
using OfficeOpenXml.Export.PdfExport;
using OfficeOpenXml.Interfaces.Fonts;
using OfficeOpenXml.Export.PdfExport.Data;
using OfficeOpenXml.Export.PdfExport.Layout;
using OfficeOpenXml.FormulaParsing.Excel.Functions.Information;
using OfficeOpenXml.Export.PdfExport.Settings;
using OfficeOpenXml.FormulaParsing.Excel.Functions.Information;
using OfficeOpenXml.Interfaces.Fonts;
using OfficeOpenXml.Style;
using System.Diagnostics;
using System.Globalization;
Expand Down Expand Up @@ -530,6 +531,21 @@ public void ThreeFonts_NoSkip_RendersAllThreeCorrectly()
SaveAsPdf(ws, "ThreeFonts_NoSkip.pdf");
}

[TestMethod]
public void PdfColorIssue_WithCopiedWorksheet()
{
var helper = new ColorIssueTestHelper();
using var package = helper.CreateWorkbook();
SaveAsPdf(package.Workbook, "Allsvenskan2001.pdf");
}

[TestMethod]
public void PdfColorIssue2_WithSingleWorkbook()
{
using var package = OpenTemplatePackage("Allsvenskan2001.xlsx");
SaveAsPdf(package.Workbook, "Allsvenskan2001.pdf");
}

[TestMethod]
public void MultiSheetWorkbook()
{
Expand Down
1 change: 1 addition & 0 deletions src/EPPlus/Style/XmlAccess/ExcelGradientFillXml.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ internal ExcelGradientFillXml(XmlNamespaceManager nameSpaceManager)
{
GradientColor1 = new ExcelColorXml(nameSpaceManager);
GradientColor2 = new ExcelColorXml(nameSpaceManager);
GradientColor3 = new ExcelColorXml(nameSpaceManager);
}
internal ExcelGradientFillXml(XmlNamespaceManager nsm, XmlNode topNode) :
base(nsm, topNode)
Expand Down
Loading