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.

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

PDFテンプレートで原価項目が出力されない問題の修正 #1058

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

K-Haruto
Copy link
Collaborator

@K-Haruto K-Haruto commented Aug 3, 2024

関連Issue / Related Issue

不具合の内容 / Bug

  • PDFテンプレートで原価項目($quotes-purchase_cost$)が正常に表示されない。

テスト環境(v7.4.0)では原価の値が0と出力されるのではなく、何らかの別の値が表示されてしまうことを確認しました。
一方でv7.3.9およびv7.3.8では所望の値が表示されておりました。
もし、不具合の内容に関して私の認識に誤りがございましたら、ご指摘いただけますと幸いです。

原因 / Cause

  • PDFテンプレートで入力された項目名($quotes-purchase_cost$)とgetMergeInventoryBlock()で置換される項目名($quotes-purchasecost$、ハイフン無し)が一致していなかったため、項目が置換されるべき場所で置換されていなかった

変更内容 / Details of Change

  • プライベート関数convertFieldName()内でフィールド名を変更(purchasecost -> purchase_cost)し、getMergeInventoryBlock()内で置換されるように変更

スクリーンショット / Screenshot

  • PDFテンプレート
    image
  • 修正前の見積書(製品A: 単価¥2,000 原価¥500、製品B: 単価¥1,000 原価¥100)
    image
  • 修正後の見積書
    image

影響範囲 / Affected Area

PDFテンプレート関係

チェックリスト / Check List

  • 自らテストを行った
  • 不必要な変更が無い
  • 影響範囲の検討を行った

備考 / Remarks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[不具合] PDFテンプレートで原価項目が出力されなくなった
1 participant