Problem/Motivation

The url() function contains workarounds to allow the installer to function.
This will prevent us from safely being able to remove usages of it in #2340251: Remove most remaining url() calls.

Proposed resolution

Move the workaround code to the NullGenerator.

Remaining tasks

User interface changes

API changes

CommentFileSizeAuthor
#1 2342925-url-null-1.patch2.2 KBtim.plunkett

Comments

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Status: Active » Needs review
StatusFileSize
new 2342925-url-null-1.patch2.2 KB
dawehner’s picture

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

Looks like a nice cleanup atm.!

webchick’s picture

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

Nice. That works!

Committed and pushed to 8.x. Thanks!

  • webchick committed 5baf8db on 8.0.x 
    Issue #2342925 by tim.plunkett: Simplify url() in order for the...

Status: Fixed » Closed (fixed)

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