The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.Problem/Motivation
_bs_base_set_yml_value() function is buggy and it will fail in a case when root variable partially exists in yaml file. It should update the variable but currently, it will just add a new version and produce invalid yaml file.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | improve-handling-of-existing-yaml-values-3068227-7.patch | 3.83 KB | pivica |
| #7 | interdiff-3068227-6-7.txt | 1.44 KB | pivica |
Comments
Comment #2
pivica commentedHere is a patch.
Comment #3
sasanikolic commentedTested and works as expected with the patch.
Comment #4
pivica commentedWhile debugging another issue i found out that regexp is not catching all possible variations, here is a new patch.
Comment #5
pivica commentedAnd now i figure it that / delimiter is not good because it's in path also, lets try #.
Comment #6
pivica commentedThe wrong patch attached in last comment, here is correct one.
Comment #7
pivica commentedImproved comment a bit and added exception throwing if you try to use anything else then primitive values.
Comment #9
pivica commentedCommitted.