This is the only method that somehow got missed in the epic conversion to camelCasing for views methods. We can't leave one method!

CommentFileSizeAuthor
#4 2056943.vdc_.renderLink.4.patch28.8 KBalexpott
#4 2056943.vdc_.renderLink.interdiff.txt1.88 KBalexpott
vdc.renderLink.patch28.36 KBdamiankloip

Comments

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Issue tags: +Approved API change

I'm comfortable calling this an approved API change. The rest of this renaming initiative was done in time for code freeze, so it makes sense to finish what we started here.

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Reviewed & tested by the community
Issue tags: -Approved API change

Perfect!

dawehner’s picture

dawehner
Primary language German
commented
Issue tags: +Approved API change
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new 2056943.vdc_.renderLink.interdiff.txt1.88 KB
new 2056943.vdc_.renderLink.4.patch28.8 KB

Hmmm, you missed some comment changes.

damiankloip’s picture

damiankloip commented
Status: Needs review » Reviewed & tested by the community

Great. Looks good.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Ok, great. Committed and pushed to 8.x. Thanks!

Looks like the change notice for this is happening at #1856630: [Change notice] [META] Rename Views methods to core standards so I'll mark this fixed. I beg of you to please try and get that change notice polished off quickly though; I took stats last week and unwritten change notices account for something like 25% of the total critical issues. :(

Automatically closed -- issue fixed for 2 weeks with no activity.