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

Fix exporter name type #21057

Merged
merged 1 commit into from May 20, 2023
Merged

Fix exporter name type #21057

merged 1 commit into from May 20, 2023

Conversation

paurosello
Copy link
Contributor

@paurosello paurosello commented May 20, 2023

This PR fixes CSV/Excel exporting Child Tables when the Document name is set to Autoincrement.

With autoincrement, the type of the name is casted to integer and the dict key is always a string, which caused exporter to always return empty Child Tables.

In order to reproduce a simple document with autoincrement can be created with a Child Table and try exporting all data.

@github-actions github-actions bot added the add-test-cases Add test case to validate fix or enhancement label May 20, 2023
@codecov
Copy link

codecov bot commented May 20, 2023

Codecov Report

Merging #21057 (3e6ce67) into develop (a0fe38e) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #21057      +/-   ##
===========================================
- Coverage    63.95%   63.95%   -0.01%     
===========================================
  Files          764      764              
  Lines        69043    69043              
  Branches      6245     6245              
===========================================
- Hits         44156    44153       -3     
- Misses       21339    21342       +3     
  Partials      3548     3548              
Flag Coverage Δ
server 68.92% <100.00%> (-0.02%) ⬇️

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

@phot0n phot0n merged commit 5394e73 into frappe:develop May 20, 2023
18 of 20 checks passed
@phot0n phot0n added backport version-14-hotfix backport to version 14 and removed add-test-cases Add test case to validate fix or enhancement labels May 20, 2023
mergify bot pushed a commit that referenced this pull request May 20, 2023
surajshetty3416 added a commit that referenced this pull request May 22, 2023
frappe-pr-bot pushed a commit that referenced this pull request May 23, 2023
## [14.36.2](v14.36.1...v14.36.2) (2023-05-23)

### Bug Fixes

* `get_default_address` must accept `'Nonetype' name` (backport [#21022](#21022)) ([#21036](#21036)) ([b10a65d](b10a65d))
* add card shadow to onboarding widget ([#21031](#21031)) ([#21032](#21032)) ([95538a2](95538a2))
* allow custom html block based on roles applied ([ad86bb9](ad86bb9))
* backend to create workspace custom block and update workspace ([3583490](3583490))
* **build:** Propogate exit code from yarn build to bench build ([#21084](#21084)) ([#21086](#21086)) ([8a14c42](8a14c42))
* cast doc name to string while exporting child docs ([#21057](#21057)) ([e57f420](e57f420))
* created custom block widget which renders html,css,js from custom html block ([d46a7c9](d46a7c9))
* created workspace custom block doctype to store custom blocks ([d08401d](d08401d))
* custom block css for widget-head ([13b91ea](13b91ea))
* custom html block ([bbeaa9e](bbeaa9e))
* frappe.get_palette when text is undefined ([#21033](#21033)) ([#21035](#21035)) ([fdcb373](fdcb373))
* handle block width for min-width = 2 ([e467254](e467254))
* incorrect caching of `is_owner` permissions ([#21048](#21048)) ([#21053](#21053)) ([00da02b](00da02b))
* initialize custom block in editorjs ([8f91cf9](8f91cf9))
* misc onboarding fixes ([#21078](#21078)) ([#21085](#21085)) ([7a754b7](7a754b7))
* read-only form for multiple role access in the same workflow state ([#21008](#21008)) ([b4ffb62](b4ffb62))
* remove setup_complete events ([5870e54](5870e54))
* sync workspace custom block doctype while installing ([4853897](4853897))
* using 'Equals' and 'Is' on multiselect throws error ([e5bd19d](e5bd19d))
* **UX:** activate next step automatically ([da3f566](da3f566))
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport version-14-hotfix backport to version 14
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants