Problem/Motivation

At #3571596-7: Remove authorize.php related Twig and CSS from stable9, @dcam pointed out that there's 1 more lingering reference to "authorize.php" in our code:

https://git.drupalcode.org/project/drupal/-/blob/main/core/lib/Drupal/Co...

Seems bad scope to remove that in the other issue, so I'm splitting it off to here.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3575873

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

dww created an issue. See original summary.

dww opened merge request !14919

dww’s picture

dww
we/he/they
commented
Status: Active » Needs review

MR is up. I credited @dcam at the contribution record.

dcam’s picture

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

It took me a few minutes to verify my earlier assertion that this is the last reference to authorize.php. PHPStorm's search index hadn't updated properly and was reporting different results from grep. I had to chase down phantom search results, then restart to refresh it.

Anyway, this really is it from what I can tell (aside from the stable9 theming removal issue). This looks good to me.

dww’s picture

dww
we/he/they
commented

I also did some manual recursive grep and also didn't find any more references...

catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to main, thanks!

Well done both adding and removing the entire subsystem!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

catch closed merge request !14919

  • catch committed ff7af288 on main 
    task: #3575873 Remove final authorize.php reference in comment

By: dww...

Status: Fixed » Closed (fixed)

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