Closed (cannot reproduce)
Project:
Drupal core
Version:
8.9.x-dev
Component:
install system
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
11 Nov 2011 at 04:31 UTC
Updated:
15 Jul 2020 at 15:35 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
David_Rothstein commentedThe first patch contains tests only and should have two failures. The second patch also contains the fix and should pass.
While writing this patch, it occurred to me that the fact that install_profile_info() even takes a langcode as the second parameter is a little suspect (the way it's used it seems like it would be better as separate logic in the installer rather than hardcoded in an API function). But that's only half the bug anyway, and we can't remove this parameter here if we want to backport this fix to Drupal 7, so I've fixed both issues for the time being. I may open a followup about removing that parameter in Drupal 8, though.
Comment #2
kscheirer#1: install-profile-info.patch queued for re-testing.
Comment #4
cilefen commentedThis is possibly a duplicate of #2474537: Installing in a non-English language fails for command-line installations (Drush, ...).
Comment #5
cilefen commentedI changed the scope of this issue and setting back to active because of the time passed and because #2474537: Installing in a non-English language fails for command-line installations (Drush, ...) will take care of the caching behavior.
Comment #6
cilefen commentedComment #15
simeI started stubbing out the new test using KernelTestCase (the old one using DrupalWebTestCase).
The old test loads three sets of profile data.
I couldn't see any discrepancy in the returned data, and it's not clear what the discrepancy should look like. One of the assertions check that the locale module is set as a dependency of the spanish one, and it appears to be fine. So I guess I can't reproduce and I want to assume that something is fixed in the last 9 years.
The raw array data from the above code is below.