fix: better check of whether doctype is child table #19329
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #19329 +/- ##
===========================================
- Coverage 64.68% 64.63% -0.06%
===========================================
Files 754 754
Lines 70978 71006 +28
Branches 6117 6117
===========================================
- Hits 45915 45896 -19
- Misses 21562 21609 +47
Partials 3501 3501
Flags with carried forward coverage won't be shown. Click here to find out more. |
90f53b7 to
aeab7cc
Compare
aeab7cc to
ce4b940
Compare
ce4b940 to
6e318dd
Compare
e225f8a to
2bb1721
Compare
ankush
approved these changes
Dec 19, 2022
mergify bot
pushed a commit
that referenced
this pull request
Dec 19, 2022
* fix: better check of whether doctype is child table * perf: init `_table_fields` earlier (cherry picked from commit 2fd2d42)
ankush
pushed a commit
that referenced
this pull request
Dec 19, 2022
frappe-pr-bot
pushed a commit
that referenced
this pull request
Dec 20, 2022
# [14.20.0](v14.19.1...v14.20.0) (2022-12-20) ### Bug Fixes * allow zero page length in `get_list` to return complete list ([#19322](#19322)) ([#19341](#19341)) ([715aaa6](715aaa6)) * better check of whether doctype is child table ([#19329](#19329)) ([#19343](#19343)) ([0c0d1ef](0c0d1ef)) * datepicker not working for arabic language (backport [#19358](#19358)) ([#19361](#19361)) ([8d791cc](8d791cc)) * db ssl connection ([#19308](#19308)) ([#19339](#19339)) ([3bf65be](3bf65be)) * dont share with self for standard users (backport [#19327](#19327)) ([#19338](#19338)) ([a3be135](a3be135)) * duplicate unique indexes (backport [#19291](#19291)) ([#19297](#19297)) ([bc92e1c](bc92e1c)) * handle `HTML` code field's `has_content` ([9bc679e](9bc679e)) * if grid custom button is not set grid has some extra space on top ([4f19f68](4f19f68)) * in web form if print format is empty getting error ([f8e227c](f8e227c)) * move filelock import to function ([ee9b1aa](ee9b1aa)) * postgres query ([#19357](#19357)) ([#19359](#19359)) ([12ecae2](12ecae2)) * **postgres:** psql with remote databases (backport [#19362](#19362)) ([#19366](#19366)) ([60c853d](60c853d)) * restore missing `helpful` and `not_helpful` fields to help article doctype. ([#19309](#19309)) ([#19316](#19316)) ([3950865](3950865)), closes [#diff-3e0e1befb66af4c2ab84c6fe0e996dc602d906817f0b00b6239120af9da9be46](https://github.com/frappe/frappe/issues/diff-3e0e1befb66af4c2ab84c6fe0e996dc602d906817f0b00b6239120af9da9be46) * sort prepared report filters ([#19267](#19267)) ([b3e16b8](b3e16b8)) * **trim-database:** Dont act on non-frappe tables (backport [#19292](#19292)) ([#19298](#19298)) ([34b2b24](34b2b24)) * use stricter regex for `sanitize_searchfield` (backport [#19277](#19277)) ([#19303](#19303)) ([4e870b3](4e870b3)) * **workflow:** Remove docstatus field from get_workflow_state_count ([#19332](#19332)) ([#19344](#19344)) ([34e8ecb](34e8ecb)) ### Features * add parameters language and letter_head to `download_pdf` ([99631d2](99631d2)) * fields for attaching (html) meta tags in web form (backport [#19306](#19306)) ([#19307](#19307)) ([6b0e585](6b0e585)) ### Performance Improvements * reuse `_doc_before_save` in `doc.get_latest` (backport [#19326](#19326)) ([#19346](#19346)) ([ff3de82](ff3de82))
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is more reliable.