- 🇺🇸United States samc
Backlink to open flexinode issue: http://drupal.org/node/42762
- 🇨🇦Canada chx
(21:46:24) vertice: the problem is that during the upload process, the stuff gets removed OUT of the form system
(21:46:28) vertice: and gets stuffed in the session
(21:46:36) vertice: and every file api module does this on it's ownThis is not the place neither the time to fix file API. Everybody on its own. I am outta here. If the change gets in, I will doc the change.
- 🇩🇪Germany killes@www.drop.org
I am not too happy about it, but unless somethign better comes up, I will commit it.
- 🇳🇱Netherlands Bèr Kessels
This works. It fixes the issue.
but:
* Its a HACK, spelled in capitals all along. Whether or not we allow such a hack to get in to solve this issue, or not, is not up to me.
* Will it not open up a bit of a security hole? I mean: if we are making exceptions that will allow certain "things" not to pass by the validation, we are moving on slippery ice (or whatever the saying is).but:
again: it solves the issue. Tested, and confirmed with a few flexinode file fields.thanks. but I dare not give it a + or - 1. Such issues, IMO are for Dries to decide.....
- 🇳🇱Netherlands Bèr Kessels
I forgot to mention, that there are other ways around this issue. For example image.module does this. They are not solutions, but simply ways to not run into this form api bug.
* do not set the required flag on a file field.
* then validate it yourself in a validation hook.The downside is, that the "required" mark (by default the *) is not printed on the file form.
- 🇩🇪Germany killes@www.drop.org
Maybe we should simply document that you can't use required with file fields. Also, Ber coul dpass an attribute to his home-brew validation routine.
Since this issue does not affect core, I'd like to not patch core, but to simply update the docs. Opinions?
- 🇺🇸United States samc
Like the previous posters, I'm not crazy about the idea of introducing such a hack into core.
I may have found a potential workaround, at least as far as flexinode goes (but should be generally applicable).
Can you guys take a look at http://drupal.org/node/42762, #12. It seems to work fine... Am I missing something that makes this a bad approach?
It might be better to ensure that that type of approach is supported via forms system, rather than just bypass required checking for file fields.
What do you think?
- 🇺🇸United States moshe weitzman Boston, MA
I'm OK with this patch for 4.7.0. We can make it work properly for 4.7.1 IMO
- 🇨🇦Canada chx
We agreed that the solution is that I am doccing right now that #type file can't be required.
- 🇳🇱Netherlands Bèr Kessels
I am sure we agreed to lower the priority and move this to the "file system" component. "wont fixes" will be forgotton. This should not be forgotton.
For it remains a bug.
- 🇨🇦Canada nedjo
I marked http://drupal.org/node/64984 as a duplicate. There is a patch on that issue.
- 🇺🇸United States neclimdul Houston, TX
Patch does not apply and a conversion of the patch does not seem to work.
- 🇺🇸United States bomarmonk
Is a patch applied for 4.7? It seems that is should be, as flexinode is still better at handling files and many sites are using it. This seems like a sad oversight... apply a patch to core that fixes this, please (easy for me to say, since I'm more of a bug hunter than a bug fixer). Just throwing my support at this as loudly as possible.
- 🇨🇦Canada webchick Vancouver 🇨🇦
This is still an issue in 5.x and, presumably, HEAD.
This prevents programmatically creating node content with drupal_execute on imagefields, for example, when the field's #required property is set to TRUE.
- 🇺🇸United States ilmaestro
...still an issue.
I'm sure it goes without saying, but stating that the #required setting is not allowable for file types is a pretty lousy solution.
- 🇬🇧United Kingdom jakeg
it may be a lousy option to say #required is not allowable for file types, but at the moment #required doesn't work for them and I've just spent hours trying to figure out where in my script I was going wrong not getting it working. I had already done my own workaround (also suggested above) of having my own #validate function and doing stuff in there, and not having the field #required, but the fact that the docs say #required is allowed is very confusing. For now at least, the docs must be changed to make #required not an option for 'file' form fields, otherwise other coders will face the same headache in figuring out where they've gone wrong in implementing it.
- 🇨🇦Canada webchick Vancouver 🇨🇦
@jakeg: Please update the docs. Anyone with a CVS account may do so. The file's in contributions/docs/developer/forms_api_reference.html, I believe.
- 🇬🇧United Kingdom jakeg
@webchick: thanks, I didn't realise/think I had access. Have updated the docs:
http://cvs.drupal.org/viewvc.py/drupal/contributions/docs/developer/topi... - 🇺🇸United States celstonvml
Subscribing
Patch from post #13 appears to be working in my 5.7 instance.
- 🇺🇸United States quicksketch
Ha, this was "fixed" 3 years(!) ago in #64984-4: File Uploads fail when field is set to "required" → , but it never made it into core. Sad, sad. Here's a reroll of my old patch, not yet tested, but just putting this on the radar.
- 🇫🇷France yonailo Paris
@jakeg: thanks for fixing up the documentation, it was really important IMHO and I have experienced this headache so I totally agree with #18
- 🇺🇸United States moshe weitzman Boston, MA
Fixes a bug that has pestered since FAPI 1.0. A test would be nice but squashing the bug is more important.
- 🇺🇸United States drewish
i'd disagree since the whole point of the test is making sure it says unbroken.
- 🇨🇦Canada webchick Vancouver 🇨🇦
Wow!! Great to see this bug fixed! But I agree that a test is mandatory.
Also, is there by chance a nicer way to get this value than $elements['#parents'][0]? that's not really at all intuitive.
- 🇺🇸United States sreynen
On testing, the previous patch went too far and didn't throw an error when a required file field was actually missing a file. Updated patch, including a test.
- 🇺🇸United States moshe weitzman Boston, MA
@Scott - the bot tested today and this patch will not apply cleanly :(
- 🇷🇺Russia vectoroc
Increase priority. Patch looks good.
Don't get why it still not fixed for years - 🇺🇸United States kleinmp
Here's the patch re-roled against the newest drupal 7 dev.
The last submitted patch, file-field-required-59750-39.patch, failed testing.
- 🇫🇷France Damien Tournoud
This is going to need to be fixed in Drupal 8 first.
- 🇺🇸United States kleinmp
I re-roled the patch off of drupal 8 and hopefully fixed the simple test issue.
- 🇨🇦Canada marcingy
So we have a test for this already in file.test for the file module so I actually assume this is fixed in core....as a result closing as this is fixed.
- 🇺🇸United States dave reid Nebraska USA
No, this is fixed for file_managed fields, but not for #type => 'file'. This is still a valid bug in D8 and below.
- 🇺🇸United States quicksketch
Yeah the #type = managed_file element has mitigated this problem significantly, since most uploads would want to use the more advanced managed_file element anyway (who wouldn't want an AJAX-upload and progress bar?) I personally don't think this is a "major" issue any more and it's not affecting nearly as many sites now since File module is available in core to provide the majority use-case. This seems like just an inconvenience now.
- 🇺🇸United States dave reid Nebraska USA
For some uses managed_file is not appropriate, but yeah I agree with the bump down to normal.
- 🇮🇳India fromtheindia
#39: file-field-required-59750-39.patch queued for re-testing.
- 🇺🇸United States kscheirer Vallejo
#42: file-field-required-59750-42.patch queued for re-testing.
The last submitted patch, file-field-required-59750-42.patch, failed testing.
Drupal 8.0.6 → was released on April 6 and is the final bugfix release for the Drupal 8.0.x series. Drupal 8.0.x will not receive any further development aside from security fixes. Drupal 8.1.0-rc1 → is now available and sites should prepare to update to 8.1.0.
Bug reports should be targeted against the 8.1.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.2.x-dev branch. For more information see the Drupal 8 minor version schedule → and the Allowed changes during the Drupal 8 release cycle → .
- 🇳🇱Netherlands rudiedirkx
This patch doesn't keep in mind
#name
changes. In a file input it's very useful to be able to name it to "files[photos][]" so you can upload multiple. (Yes, HTML has been doing that for a couple of years now.) It also doesn't handle files in a fieldset with#tree
=> TRUE.I've solved this differently: http://drupalcode.org/project/value_is.git/blob/refs/heads/7.x-1.x:/valu...
Drupal 8.8.7 → was released on June 3, 2020 and is the final full bugfix release for the Drupal 8.8.x series. Drupal 8.8.x will not receive any further development aside from security fixes. Sites should prepare to update to Drupal 8.9.0 → or Drupal 9.0.0 → for ongoing support.
Bug reports should be targeted against the 8.9.x-dev branch from now on, and new development or disruptive changes should be targeted against the 9.1.x-dev branch. For more information see the Drupal 8 and 9 minor version schedule → and the Allowed changes during the Drupal 8 and 9 release cycles → .
Drupal 8.6.x will not receive any further development aside from security fixes. Bug reports should be targeted against the 8.8.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.9.x-dev branch. For more information see the Drupal 8 and 9 minor version schedule → and the Allowed changes during the Drupal 8 and 9 release cycles → .
Drupal 8.5.6 → was released on August 1, 2018 and is the final bugfix release for the Drupal 8.5.x series. Drupal 8.5.x will not receive any further development aside from security fixes. Sites should prepare to update to 8.6.0 on September 5, 2018. ( Drupal 8.6.0-rc1 → is available for testing.)
Bug reports should be targeted against the 8.6.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.7.x-dev branch. For more information see the Drupal 8 minor version schedule → and the Allowed changes during the Drupal 8 release cycle → .
Drupal 8.4.4 → was released on January 3, 2018 and is the final full bugfix release for the Drupal 8.4.x series. Drupal 8.4.x will not receive any further development aside from critical and security fixes. Sites should prepare to update to 8.5.0 on March 7, 2018. ( Drupal 8.5.0-alpha1 → is available for testing.)
Bug reports should be targeted against the 8.5.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.6.x-dev branch. For more information see the Drupal 8 minor version schedule → and the Allowed changes during the Drupal 8 release cycle → .
Drupal 8.3.6 → was released on August 2, 2017 and is the final full bugfix release for the Drupal 8.3.x series. Drupal 8.3.x will not receive any further development aside from critical and security fixes. Sites should prepare to update to 8.4.0 on October 4, 2017. ( Drupal 8.4.0-alpha1 → is available for testing.)
Bug reports should be targeted against the 8.4.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.5.x-dev branch. For more information see the Drupal 8 minor version schedule → and the Allowed changes during the Drupal 8 release cycle → .
Drupal 8.2.6 → was released on February 1, 2017 and is the final full bugfix release for the Drupal 8.2.x series. Drupal 8.2.x will not receive any further development aside from critical and security fixes. Sites should prepare to update to 8.3.0 on April 5, 2017. ( Drupal 8.3.0-alpha1 → is available for testing.)
Bug reports should be targeted against the 8.3.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.4.x-dev branch. For more information see the Drupal 8 minor version schedule → and the Allowed changes during the Drupal 8 release cycle → .
Drupal 8.1.9 → was released on September 7 and is the final bugfix release for the Drupal 8.1.x series. Drupal 8.1.x will not receive any further development aside from security fixes. Drupal 8.2.0-rc1 → is now available and sites should prepare to upgrade to 8.2.0.
Bug reports should be targeted against the 8.2.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.3.x-dev branch. For more information see the Drupal 8 minor version schedule → and the Allowed changes during the Drupal 8 release cycle → .
- 🇦🇺Australia kim.pepper 🏄♂️🇦🇺Sydney, Australia
This needs a re-roll for 8.9.x.
- 🇳🇿New Zealand quietone
A file form, that is required will always fail.
Does this mean a form with a file field that is required?
- 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
Discussed this with @quietone
\Drupal\file_test\Form\FileTestForm
provides a sample form and\Drupal\Tests\file\Functional\SaveUploadTest
tests it.If we make the element there #required we should yield the bug.
I think we can solve this differently now that we have plugins for elements.
We can put a valueCallback on the element and populate it with the temporary file name from the request's file bag if a file with that name has been uploaded.
- 🇳🇿New Zealand quietone
Thanks to @larowlan for proving the sample code for this in #bugsmash. With that I have made a rough patch as well as a fail patch. There is no interdiff because this is a different approach.
This makes the file field required for all the tests using FileTestForm. I haven't looked yet but is there a still a test without the field being required?
- 🇳🇿New Zealand quietone
+++ b/core/lib/Drupal/Core/Render/Element/File.php @@ -72,4 +75,23 @@ public static function preRenderFile($element) { + * @param array $element + * @param mixed $input + * @param \Drupal\Core\Form\FormStateInterface $form_state + * + * @return mixed|null
Should be @inheritdoc
- 🇳🇱Netherlands Lendude Amsterdam
Yeah an probably better
+++ b/core/modules/file/tests/file_test/src/Form/FileTestForm.php @@ -24,6 +24,7 @@ public function getFormId() { '#type' => 'file', + '#required' => TRUE, '#title' => t('Upload a file'),
Yeah and probably better/easier to split the required field off into its own form that you can use just for testing that
- 🇳🇿New Zealand quietone
OK.
I removed the addition of required in FileTestForm and added a new test form. Instead of making a new test file I added, testRequired, to the existing SaveUploadTest.php although I am not sure if that is the best way to do this.
The last submitted patch, 77: 59750-77.patch, failed testing. View results →
- 🇳🇿New Zealand quietone
Not sure if there is a better way, this now checks that the files variable is of the UploadedFile class before treating it like an object.
The last submitted patch, 79: 59750-79.patch, failed testing. View results →
- 🇳🇿New Zealand quietone
Restore the test that $all_files is not empty. Don't know why I removed it. Added comments to the test.
The last submitted patch, 81: 59750-81.patch, failed testing. View results →
- 🇳🇿New Zealand quietone
Changing to use NULL coalesce on the file variable.
- 🇧🇷Brazil paulocs Belo Horizonte
Test looks good and the problem is fixed.
I'm adding in the IS the steps to reproduce. - 🇳🇿New Zealand quietone
I'm adding credit to larowlan who provided the solution, and support, in bugsmash.
- 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
+ $image2 = current($this->drupalGetTestFiles('image')); + $edit = ['files[file_test_upload]' => \Drupal::service('file_system')->realpath($image2->uri)];
Nit, should this be image instead of images?
- 🇳🇿New Zealand quietone
The new test uses code from SaveUploadTest::testNormal and SaveUploadTest::testDuplicate and both of those use $image2 for the second image that is uploaded. For example, https://git.drupalcode.org/project/drupal/-/blob/9.3.x/core/modules/file.... I thought it best to kept to the existing style in the file.
Happy to change it though, shall I?
- 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
I think with it as image2 our future selves might wonder what happened to image 1, so its probably worth fixing for future comprehension sake
can be fixed on commit though
- 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
I'm going to ask chx on slack if he can have a look at this to make sure we're not doing anything wrong in the value callback
- 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
Discussed this with @chx on slack and he pointed out a few minor code-style changes
But we thought that using #value for this might be risky, because file elements have never had a value.
So I've come up with a different approach, patch to follow
The last submitted patch, 93: 59750-94.patch, failed testing. View results →
- 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
So I've thought about this some more and I think the approach of unsetting #required is wrong - as the tests show - because it means that the required state is not available to the theme system
So I think we should go back to the approach from #83, here's that patch plus some minor changes that chx identified on slack
Because this will mean that file elements now have a #value, we'll need a change record.
Updated the issue summary while here.
Thinking about this some more, should we put the UploadedFile object in the value instead of just the file name?
The last submitted patch, 95: 59750-95.patch, failed testing. View results →
- 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
Ok, those fails have convinced me, because with #multiple this needs to be an array.
So I'll make the #value an array of UploadedFile objects
The last submitted patch, 98: 59750-98.patch, failed testing. View results →
- 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
Ok, that fail seems to point to somewhere we're using #value for a file element, in theme settings. So this is good, we might have found a possible regression
- 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
So theme settings form has a #maxlength on a file upload field, which obviously makes no sense.
Removing it.
- 🇺🇸United States neclimdul Houston, TX
"Why is this in my dashboard? I don't remember running into this recently that's just the way things work. ... Oh, I commented _15 years ago_!"
This isn't so much a review of the patch but an observation about the form api.
+ public static function valueCallback(&$element, $input, FormStateInterface $form_state) { + if ($input === FALSE) { + return NULL; + } + $element_name = implode('_', $element['#parents']); + $uploaded_files = \Drupal::request()->files->get('files', []); + $uploaded_file = $uploaded_files[$element_name] ?? NULL;
This smells bad. Why do we need to look at the request? Makes something already difficult to test(I'm waiting on a test run for something I'm trying to write while writing this) and likely makes it harder. I know the answer is not the fault of this patch its just the only option. Should the FormState give us access to files the same way it does values? Probably. Is that likely to be hacky and ugly which is why it doesn't exist? Again probably but worth the effort in another patch/follow up I think.
$element_name = implode('_', $element['#parents']);
This part was also confusing until I confirmed it matches logic duplicated in a couple places in file.module. Again not a problem with this patch but highlighting maybe some debt in FormApi's file handling that could be addressed. Again in formstate's interface?
This isn't going to be backported to 6 or 7 at this point I wager so a little tag cleanup as well.
Patch looks solid and skim of tests looks good. Wager this is good to go but smell makes me feel like it needs more manually testing than I can do in this review so leaving an rtbc to someone else or another review.
- 🇦🇺Australia kattekrab
Thanks for the change record @ quietone → - Looks good. I'll remove the needs CR tag now.
I'm going to have a crack at a manual test, but it's been a while since I've done this. Wish me luck ;-)Reading the issue thread here is like a who's who of Drupal!
It would be amazing to get this one smashed. :) - 🇺🇸United States mikeryan Murphysboro, IL, USA
Not that anyone asked for this... But I'm working on a D7 project now that needs this, so here's a reroll of #42.
The last submitted patch, 109: file-field-required-59750-109.patch, failed testing. View results →
- 🇺🇸United States mikeryan Murphysboro, IL, USA
Restoring "Needs review", since my D7 testing bumped it back to "Needs work"...
I have an issue with a part of the patch :
implode('_', $element['#parents'])
This part does not work for me because i use sub index in my form for file type (#tree = TRUE).
If i replace this by :
array_shift($element['#parents'])
It works for me. What do you think about this?
https://www.drupal.org/files/issues/2021-11-30/59750-112.patch →Drupal 9.4.0-alpha1 → was released on May 6, 2022, which means new developments and disruptive changes should now be targeted for the 9.5.x-dev branch. For more information see the Drupal core minor version schedule → and the Allowed changes during the Drupal core release cycle → .
Drupal 9.3.0-rc1 → was released on November 26, 2021, which means new developments and disruptive changes should now be targeted for the 9.4.x-dev branch. For more information see the Drupal core minor version schedule → and the Allowed changes during the Drupal core release cycle → .
- 🇳🇿New Zealand quietone
The patch in #116 still applies to 9.5.x.
Adding interdiff.
- 🇦🇺Australia kim.pepper 🏄♂️🇦🇺Sydney, Australia
+++ b/core/lib/Drupal/Core/Render/Element/File.php @@ -72,4 +79,23 @@ public static function preRenderFile($element) { + $uploaded_files = \Drupal::request()->files->get('files', []);
Do we have an issue to move files to form state instead of calling this?
I think this needs a test-only patch.
The last submitted patch, 121: 59750-121-test-only.patch, failed testing. View results →
- 🇳🇿New Zealand quietone
Updating for 10.0.x and adding interdiffs. Interdiff.txt is between the success and fail patch. The other interdiff is highlight the changed suggested in #114 since there was not an interdiff in #116.
The last submitted patch, 123: 59750-123-fail.patch, failed testing. View results →
- 🇻🇳Vietnam mrddthi
The patches
- `59750-123.patch` #123
- `file-field-required-59750-114.patch` #114
- `59750-101.patch` #111
could be applied via composer but the feature still not work
Attached the image for your reference
- 🇳🇿New Zealand quietone
@mrddthi, thank you for your interest in this issue. What patch did you test? Your comment lists three patches.
I decided to retest with #123 on Drupal 10. It works fine for me, I even managed to make a screencast.
- 🇺🇸United States smustgrave
Tested using the form api sub module from the examples module
Added the file type to a simple form confirmed the bug.
Applied the patch and the issue was resolved.
On Drupal 9.5.x
The last submitted patch, 123: 59750-123.patch, failed testing. View results →
- 🇳🇿New Zealand quietone
Test failure in unrelated file
1) Drupal\KernelTests\Core\Entity\ConfigEntityQueryTest::testCaseSensitivity Failed asserting that actual size 4 matches expected size 3. /var/www/html/vendor/phpunit/phpunit/src/Framework/Constraint/Constraint.php:121 /var/www/html/vendor/phpunit/phpunit/src/Framework/Constraint/Constraint.php:55 /var/www/html/core/tests/Drupal/KernelTests/Core/Entity/ConfigEntityQueryTest.php:734 /var/www/html/core/tests/Drupal/KernelTests/Core/Entity/ConfigEntityQueryTest.php:658 /var/www/html/vendor/phpunit/phpunit/src/Framework/TestResult.php:726
then I found #3101130: ConfigEntityQueryTest::testCaseSensitivity can randomly fail → . Maybe related?
Starting tests again.
- 🇺🇸United States smustgrave
#123 seemed to pass and test-only patch failed.
The last submitted patch, 123: 59750-123.patch, failed testing. View results →
- 🇺🇸United States smustgrave
Moving back as it seemed to be a random failure.
- 🇬🇧United Kingdom alexpott 🇪🇺🌍
Committed and pushed a5b44e8f74 to 10.1.x and 964a3f3aeb to 10.0.x and e3930ea53e to 9.5.x. Thanks!
Congrats to bugsmash for resurrecting and fixing this one. Always fun to commit a 5 digit NID :D!
-
alexpott →
committed a5b44e8 on 10.1.x
Issue #59750 by quietone, larowlan, mikeryan, quicksketch, sreynen,...
-
alexpott →
committed a5b44e8 on 10.1.x
-
alexpott →
committed 964a3f3 on 10.0.x
Issue #59750 by quietone, larowlan, mikeryan, quicksketch, sreynen,...
-
alexpott →
committed 964a3f3 on 10.0.x
-
alexpott →
committed e3930ea on 9.5.x
Issue #59750 by quietone, larowlan, mikeryan, quicksketch, sreynen,...
-
alexpott →
committed e3930ea on 9.5.x
- 🇺🇸United States papagrande US West Coast
Maybe I missed this, but did the D7 patch in #111 also get committed?
- 🇬🇧United Kingdom alexpott 🇪🇺🌍
@PapaGrande the process for backporting this to D7 would be to open another issue against Drupal 7 for this. I think this change is unlikely to make it D7 at this point but that is for a D7 maintainer to decide on the new issue. Also the patch in #111 is a very different solution from the final patch.
Automatically closed - issue fixed for 2 weeks with no activity.
Just a note that this commit actually introduces a pretty big bug when the File form element has
#multiple
set tofalse
.Attempting to cast a complex object like this:
$uploaded_file = new \Symfony\Component\HttpFoundation\File\UploadedFile('/real/path.txt', 'path.txt', 'text/plain'); var_dump((array) $uploaded_file);
Will result in just the UploadedFile properties being outputed as the output array (https://stackoverflow.com/a/4345609).
The actual return value should be something along the lines of:
--- return (array) $uploaded_file; +++ return is_array($uploaded_file) ? $uploaded_file : [$uploaded_file];
- 🇦🇺Australia yovince Melbourne
yes, having some issues.
I have to set #multiple set to false, otherwise, I will get the error below
Exception: Serialization of 'Symfony\Component\HttpFoundation\File\UploadedFile' is not allowed in serialize() (line 157 of /app/docroot/core/lib/Drupal/Core/Batch/BatchStorage.php).
- 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
@codebymikey did you happen to open a new issue for that regression?
@larowlan yes, the regression issue has already referenced this issue, but I'll add a direct relation now.