Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Please find the attach screenshot. This happens when the batch process is running.
I think there should be validate function to tackle this issue
Comment | File | Size | Author |
---|---|---|---|
#14 | interdiff.txt | 1.49 KB | willzyx |
#14 | a_fatal_error_occurred-2755781-14.patch | 3.98 KB | willzyx |
| |||
#13 | devel_generate_2755781_missing_bundle_for_entity_type_test_case_13.patch | 4.07 KB | Chandan Chaudhary |
| |||
#10 | a_fatal_error_occurred-2755781-10.patch | 3.1 KB | willzyx |
| |||
#7 | devel_generate_2755781_missing_bundle_for_entity_type_7.patch | 878 bytes | Chandan Chaudhary |
|
Comments
Comment #2
Chandan Chaudhary CreditAttribution: Chandan Chaudhary as a volunteer and commentedPlease find the attach patch.
Comment #3
Chandan Chaudhary CreditAttribution: Chandan Chaudhary as a volunteer and at Therefore commentedIgnore patch 'devel_generate_2755781_missing_bundle_for_entity_type_2.patch' it has missing leading space.
Please find the update patch as 'devel_generate_2755781_missing_bundle_for_entity_type_3.patch'
Comment #4
Chandan Chaudhary CreditAttribution: Chandan Chaudhary as a volunteer and at Therefore commentedComment #7
Chandan Chaudhary CreditAttribution: Chandan Chaudhary as a volunteer and at Therefore commentedComment #8
lussolucaJust a small typo, then it's RTBC for me
Comment #9
willzyx CreditAttribution: willzyx commented@Chandan Chaudhary thanks for the patch!
IMHO the approatch used in the patch is wrong. The form validation is pugin specific (ContentDevelGenerate) and should not resides in DevelGenerateForm, it should be located in the related plugin. Also we need test for this
Comment #10
willzyx CreditAttribution: willzyx commentedSomethig like this (not tested :p) could be a good start
Comment #11
itamair CreditAttribution: itamair as a volunteer commentedComment #12
itamair CreditAttribution: itamair as a volunteer commentedComment #13
Chandan Chaudhary CreditAttribution: Chandan Chaudhary as a volunteer and at Therefore commentedi have update the patch #10 and added the test cases for this issue.
Comment #14
willzyx CreditAttribution: willzyx commentedChanged a little the test.. otherwise looks good
Comment #16
willzyx CreditAttribution: willzyx commentedCommitted and pushed to 8.x. Thanks!