- Issue created by @bingol@ciandt.com
- ๐ฎ๐ฉIndonesia gausarts
Good catch, thank you.
How about
Url::mergeOptions()
:
https://git.drupalcode.org/project/drupal/-/blob/9.4.x/core/lib/Drupal/C...Looks like less hustle and more compact in there?
Feel free to update it.
- ๐ฎ๐ณIndia jaydev bhatt Pune, Maharashtra
jaydev bhatt โ made their first commit to this issueโs fork.
- Merge request !7Issue #3534378: Fix lost attributes by using mergeOptions() instead of setOptions() โ (Merged) created by jaydev bhatt
- ๐ฎ๐ณIndia jaydev bhatt Pune, Maharashtra
Updated the method from
setOptions()
tomergeOptions()
as suggested by @gausarts to preserve existing attributes while adding theultimenu__link
class.
Moving to "Needs review" for further verification and testing. -
gausarts โ
committed ec435e2d on 3.0.x authored by
jaydev bhatt โ
Issue #3534378 by jaydev bhatt, bingol@ciandt.com: The set arrtibutes...
-
gausarts โ
committed ec435e2d on 3.0.x authored by
jaydev bhatt โ
- ๐ฎ๐ฉIndonesia gausarts
Thank you all for contributions and extra finding.