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.
I'm not sure where to submit this issue, maybe you can point me in the right direction
I've exported the variable language_default with features but everytime I clear cache the variable is changed and the feature is overriden but is not really a change in configuration.
Comment | File | Size | Author |
---|---|---|---|
#13 | interdiff.txt | 431 bytes | claudiu.cristea |
#13 | variable-1850618-13.patch | 2.25 KB | claudiu.cristea |
Comments
Comment #1
naxoc CreditAttribution: naxoc commentedI can confirm this. It is because core will change the hash-path to the js-folder, so I don't think there is much we can do about it.
For explanation, here is a diff from drush (ignore the weights - that is not the issue). This without having touched anything in the database - just installing the site on a dev-server.
Comment #2
thamasIs there a way to exclude the javascript hash-path?
Comment #3
estoyausenteSame problem, but I don't know the solution...
Comment #4
ovidenov CreditAttribution: ovidenov commentedSame problem, anyone knows a solution ?
Comment #5
rodrigoaguileraI ended up not exporting that variable, set it on install if you are installing a profile or do a hook_update_N function to set it to the desired value.
Comment #6
estoyausente@rodrigoaguilera We configure the language in the install using profiling but anyway this is a bug (or maybe a feature request).
Comment #7
stefan.r CreditAttribution: stefan.r commentedThis may be a nice to have.
Comment #8
claudiu.cristeaI see no solution other than intercepting the variables before they are rendered for export. But this means an API change. Now Strongarm allows this by implementing
hook_strongarm_export_render_alter()
.Comment #9
claudiu.cristeaOops! When such a variable is reverted using #8 I'm getting:
That means we still have to keep the
javascript
key in the object but export it everytime as empty string.Comment #10
dmsmidt#9: Nice, works for me
Comment #11
claudiu.cristeaGreat. Then why not RTBC? :)
Comment #12
dmsmidtAlmost RTBC :-)
Nice, just reading up on: https://www.drupal.org/coding-standards/docs#defgroup
So this is for http://www.drupalcontrib.org?
Add dot behind the tekst instead of /.
Comment #13
claudiu.cristea@dmsmidt,
That is used by the API module. To group various functions and classes as topics. http://www.drupalcontrib.org uses the API module so it will benefit from this docs definition but only if Strongarm module is included there. But right now is not there.
Comment #14
claudiu.cristeaComment #15
Dane Powell CreditAttribution: Dane Powell at Acquia commentedWorks great for me.
Comment #16
vasikeI can confirm the solution from #13.
Comment #17
claudiu.cristeaThen we need to let it as RTBC... maybe the module maintainer will finally commit this :)
Comment #18
gaydamaka CreditAttribution: gaydamaka commentedWhere return is not necessary, it is better to add a break.
Comment #19
RenrhafAny news about this patch being merged ?