Skip to content

Introduce 2 new functions SetCalcProps and GetCalcProps#2098

Merged
xuri merged 2 commits intoqax-os:v2from
romanshevelev:v2
Mar 4, 2025
Merged

Introduce 2 new functions SetCalcProps and GetCalcProps#2098
xuri merged 2 commits intoqax-os:v2from
romanshevelev:v2

Conversation

@romanshevelev
Copy link
Contributor

PR Details

support recalc formula cell values on load

Description

sets xl/workbook.xml/workbook/calcPr@fullCalcOnLoad=true to recalc formula cell values on Excel load workbook

Related Issue

Motivation and Context

created document with formula doesn`t calculate value on document load

How Has This Been Tested

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • [v] New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • [v] My code follows the code style of this project.
  • [v] My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • [v] I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • [v] All new and existing tests passed.

sets xl/workbook.xml/workbook/calcPr@fullCalcOnLoad=true
to recalc formula cell values on Excel load workbook
@xuri xuri added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Mar 3, 2025
- Add new CalcPropsOptions data type
- Update unit tests
@xuri xuri added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Mar 3, 2025
@xuri xuri moved this to Features in Excelize v2.9.1 Mar 3, 2025
@codecov
Copy link

codecov bot commented Mar 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.20%. Comparing base (3f6ecff) to head (7026e01).
Report is 1 commits behind head on v2.

Additional details and impacted files
@@           Coverage Diff           @@
##               v2    #2098   +/-   ##
=======================================
  Coverage   99.20%   99.20%           
=======================================
  Files          32       32           
  Lines       30007    30033   +26     
=======================================
+ Hits        29769    29795   +26     
  Misses        158      158           
  Partials       80       80           
Flag Coverage Δ
unittests 99.20% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@xuri xuri changed the title allow set fullCalcOnLoad=true attribute Introduce 2 new functions SetCalcProps and GetCalcProps Mar 4, 2025
@xuri xuri merged commit 1c0ffe3 into qax-os:v2 Mar 4, 2025
27 checks passed
xuri pushed a commit that referenced this pull request Mar 4, 2025
- Add new CalcPropsOptions data type
- Support assign exported data structure fields value to internal data structure fields dynamically by specified fields name list
- Simplify code for function SetAppProps, SetDocProps, SetWorkbookProps, GetWorkbookProps and getPivotTable
- Update unit tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

No open projects
Status: Features

Development

Successfully merging this pull request may close these issues.

2 participants

Comments