The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.| Comment | File | Size | Author |
|---|---|---|---|
| #18 | 1791442-17.patch | 19.09 KB | nick_vh |
| #12 | 1791442-12.patch | 3.16 KB | nick_vh |
| #8 | set-default-search-1791442-8.patch | 3.16 KB | cpliakas |
| #6 | 1791442-6.patch | 747 bytes | nick_vh |
| #3 | 1791442-1.patch | 516 bytes | nick_vh |
Comments
Comment #1
cpliakas commentedThe attached patch makes the suggested change. Marking as "needs work" pending SimpleTests.
Comment #2
cpliakas commentedReally marking as needs work.
Comment #3
nick_vhComment #4
nick_vhWorked as expected in manual test
Comment #5
nick_vhHmm drupal.org did weird, i missed completely Chris's posts
Comment #6
nick_vhI agree with the place of this variable-set, so it only happens once when we enable/recreate the environment
Comment #7
cpliakas commentedMarking as needs work pending tests. I have the tests working, so give me a minute to post.
Comment #8
cpliakas commentedPatch with tests.
Comment #10
nick_vh#8: set-default-search-1791442-8.patch queued for re-testing.
Comment #12
nick_vhJust re-rolled it
Comment #13
nick_vhComment #15
nick_vhLooks like the testbot fails here?
I'm ok in committing this change if local tests succeed
Comment #16
cpliakas commentedWe have line ending issues in the Acquia Search module. Ugh. Tests pass locally, let's fix the line endings.
Comment #17
cpliakas commentedSo yes, I say commit and then let's post an issue to fix the line ending issues.
Comment #18
nick_vhugh, only way to fix is like this it seems (It was windows and now back to linux)
Comment #19
nick_vhCommitted!
Comment #20
nick_vhnot applicable for 6.x-3.x
Comment #21
pwolanin commented