WIP: move get_current_url() and co in OmniTools Request class
Changes PlannedPublic
Actions

Authored by dereckson on Tue, Nov 11, 18:13.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Nov 30, 09:19

	Unknown Object (File)
	Wed, Nov 26, 18:22

	Unknown Object (File)
	Tue, Nov 25, 18:48

	Unknown Object (File)
	Mon, Nov 24, 02:37

	Unknown Object (File)
	Sun, Nov 23, 22:32

	Unknown Object (File)
	Fri, Nov 21, 22:27

	Unknown Object (File)
	Fri, Nov 21, 19:34

	Unknown Object (File)
	Fri, Nov 21, 07:15

View All 33 Files

Subscribers

None

Details

Reviewers: None

Test Plan

This code has been battle-tested with several sites using:

direct .php access in the URL
Apache + mod_rewrite
nginx try_files directive

Diff Detail

Repository

rKERUALD Keruald libraries development repository

Lint

Lint Passed

Unit

Test Failures

Branch

request-current-url

Build Status

Buildable 6184
Build 6468: arc lint + arc unit

Unit TestsBroken
View All

	Time	Test
	0 ms	/datacube/dereckson/dev/nasqueron/keruald/keruald/omnitools/tests/HTTP/Requests/RequestTest.php

Event Timeline

dereckson requested review of this revision.Tue, Nov 11, 18:13

dereckson created this revision.

Harbormaster completed remote builds in B6184: Diff 10049.Tue, Nov 11, 18:13

Unit tests still to commit

Revision Contents
Changeset List

Path

Size

omnitools/

src/

HTTP/

Requests/

Request.php

1 line

WithLocalURL.php

141 lines

Diff 10049

View Options

omnitools/src/HTTP/Requests/Request.php