Skip to content
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

Jump to bottom

Calling General Ledger->Chart of Accounts in PDF format fails to render the Credits and Debits #7739

Closed
ehuelsmann opened this issue Nov 30, 2023 · 1 comment · Fixed by #7744
Closed

Calling General Ledger->Chart of Accounts in PDF format fails to render the Credits and Debits #7739

ehuelsmann opened this issue Nov 30, 2023 · 1 comment · Fixed by #7744
Labels
type:bug Bug that will be worked on as resources permit

Comments

@ehuelsmann
Copy link
Member

Version

1.10.25

What browsers are you seeing the problem on?

This problem isn't browser related

What happened?

Calling General Ledger->Chart of Accounts in PDF format fails to render the Credits and Debits. Calling for CSV format presents the values as shown on the relevant screen.

What should have happened?

The "number" columns should have rendered.
@howardlowndes
Copy link

This problem also occurs in Reports->Trial Balance. for the :End Balance column

@neilt neilt added the type:bug Bug that will be worked on as resources permit label Dec 1, 2023
ehuelsmann added a commit to ehuelsmann/LedgerSMB that referenced this issue Dec 1, 2023
@ehuelsmann
Fix encoding of the column data, including col_id
90f2752 
Due to the fact that all column data was encoded for output-format-safety,
the CoA and Trial Balance reports didn't show some columns in PDF format
(with LaTeX intermediate representation). The column *keys* in the ROW data
do *not* get encoded, whereas the column keys (stored in the column data as
values) *do* get encoded. Due to this mismatch, the column name 'credit_balance'
differed between the row data and the column data (the latter containing the
encoded value 'credit\_balance').

Fixes ledgersmb#7739.
ehuelsmann added a commit to ehuelsmann/LedgerSMB that referenced this issue Dec 1, 2023
@ehuelsmann
Fix encoding of the column data, including col_id
6cf0bb9 
Due to the fact that all column data was encoded for output-format-safety,
the CoA and Trial Balance reports didn't show some columns in PDF format
(with LaTeX intermediate representation). The column *keys* in the ROW data
do *not* get encoded, whereas the column keys (stored in the column data as
values) *do* get encoded. Due to this mismatch, the column name 'credit_balance'
differed between the row data and the column data (the latter containing the
encoded value 'credit\_balance').

Fixes ledgersmb#7739.
ehuelsmann added a commit to ehuelsmann/LedgerSMB that referenced this issue Dec 1, 2023
@ehuelsmann
Fix encoding of the column data, including col_id
6dc1bc8 
Due to the fact that all column data was encoded for output-format-safety,
the CoA and Trial Balance reports didn't show some columns in PDF format
(with LaTeX intermediate representation). The column *keys* in the ROW data
do *not* get encoded, whereas the column keys (stored in the column data as
values) *do* get encoded. Due to this mismatch, the column name 'credit_balance'
differed between the row data and the column data (the latter containing the
encoded value 'credit\_balance').

Fixes ledgersmb#7739.
ehuelsmann added a commit to ehuelsmann/LedgerSMB that referenced this issue Dec 1, 2023
@ehuelsmann
Fix encoding of the column data, including col_id
cf970eb 
Due to the fact that all column data was encoded for output-format-safety,
the CoA and Trial Balance reports didn't show some columns in PDF format
(with LaTeX intermediate representation). The column *keys* in the ROW data
do *not* get encoded, whereas the column keys (stored in the column data as
values) *do* get encoded. Due to this mismatch, the column name 'credit_balance'
differed between the row data and the column data (the latter containing the
encoded value 'credit\_balance').

Fixes ledgersmb#7739.
This was referenced Dec 1, 2023
Merged
Merged
Merged
ehuelsmann added a commit to ehuelsmann/LedgerSMB that referenced this issue Dec 2, 2023
@ehuelsmann
Fix encoding of the column data, including col_id
806aad4 
Due to the fact that all column data was encoded for output-format-safety,
the CoA and Trial Balance reports didn't show some columns in PDF format
(with LaTeX intermediate representation). The column *keys* in the ROW data
do *not* get encoded, whereas the column keys (stored in the column data as
values) *do* get encoded. Due to this mismatch, the column name 'credit_balance'
differed between the row data and the column data (the latter containing the
encoded value 'credit\_balance').

Fixes ledgersmb#7739.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type:bug Bug that will be worked on as resources permit
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix encoding of the column data, including col_id ehuelsmann/LedgerSMB
3 participants
@neilt @ehuelsmann @howardlowndes