see patch

CommentFileSizeAuthor
htaccess_12.patch1.06 KBkilles@www.drop.org
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drewish’s picture

drewish CreditAttribution: drewish commented

+1. anyone that hasn't migrated away from those URLs should be able to figure out how to copy them in from their old .htaccess file.

chx’s picture

chx CreditAttribution: chx commented

While at it what about dropping legacy module? We have not updated it for the admin URLs in Drupal 5 and noone seemed to care.

drewish’s picture

drewish CreditAttribution: drewish commented

i'm all for it. is anyone using it? maybe we could check the data from drupal.module feed back and see if it's even being used.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Needs review » Needs work

Committed to CVS HEAD. I think it's OK to look into cleaning-up legacy.module as well. All URLs are rewritten by the upgrade scripts, and search engines should be updated by now, AFAIK.

m3avrck’s picture

m3avrck CreditAttribution: m3avrck commented
Title: clean up .htaccess » remove legacy module (was: clean up .htaccess)

Updated title to reflect task at hand.

chx’s picture

chx CreditAttribution: chx commented
Status: Needs work » Reviewed & tested by the community

There is nothing to work on here. IMO Dries could just do a cvs rm legacy.module .

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented

+1 - go for it.

BioALIEN’s picture

BioALIEN CreditAttribution: BioALIEN commented

+1 legacy module has reached EOL.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed
webernet’s picture

webernet CreditAttribution: webernet commented
Status: Fixed » Reviewed & tested by the community

legacy.info and the legacy directory need to be removed as well.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Oops. Fixed that now.

webernet’s picture

webernet CreditAttribution: webernet commented
Status: Fixed » Reviewed & tested by the community

The .info is still there... I guess that legacy just doesn't want to be removed. ;)

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Oops. Thanks. :)

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)