Part of meta-issue #1856630: [Change notice] [META] Rename Views methods to core standards

CommentFileSizeAuthor
#8 views_rename_showSortForm-2003258-8.patch690 bytesmarkie
#3 rename_showSortForm-2003258-3.patch1.09 KBmarkie

Comments

markie’s picture

markie commented
Assigned: Unassigned » markie
markie’s picture

markie commented

dibs

markie’s picture

markie commented
Status: Active » Needs review
StatusFileSize
new rename_showSortForm-2003258-3.patch1.09 KB

patched

phl3tch’s picture

phl3tch commented
Status: Needs review » Reviewed & tested by the community
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed acd9be1 and pushed to 8.x. Thanks!

istryker’s picture

istryker commented
Status: Fixed » Needs review

Shouldn't the function be public or private?

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Needs work

Right it should be protected.

markie’s picture

markie commented
Status: Needs work » Needs review
StatusFileSize
new views_rename_showSortForm-2003258-8.patch690 bytes

access updated per request.

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Reviewed & tested by the community

Thanks!

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed dd4f441 and pushed to 8.x. Thanks!

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