Context Navigation

← Previous Ticket
Next Ticket →

#10384 new enhancement

Make IIS Permalink support enabled based on capability not on version number

Reported by:	westi	Owned by:	westi
Milestone:		Priority:	normal
Severity:	normal	Version:	2.8
Component:	Permalinks	Keywords:	needs-refresh
Focuses:		Cc:

Description

At the moment we enable the IIS permalink support based on checks for IIS7 and then some capabilities.

We should remove the version checking and work solely on capabilities so that we don't have to revisit when IIS8 is released.

Attachments (2)

10384.patch (8.8 KB) - added by ruslany 15 years ago.: Patch to remove dependency on IIS version
10384-whale.patch (13.4 KB) - added by peaceablewhale 15 years ago.

Download all attachments as: .zip

Change History (15)

#1 @peaceablewhale
15 years ago

Cc peaceable_whale@… added
Type changed from defect (bug) to enhancement

@ruslany
15 years ago

Attachment 10384.patch added

Patch to remove dependency on IIS version

#2 @ruslany
15 years ago

Keywords has-patch added; needs-patch removed

The suggestion in the bug makes sense. There is no need to have a version check. It is possible to reliably figure out permalinks support on IIS even without version check.

I've attached the patch.

#3 @peaceablewhale
15 years ago

I think functions should be deprecated first before they are completely removed. In addition, some filters have to be preserved for existing plugins (if any). A new patch, based on Ruslany's patch, has been uploaded to improve backward compatibility.