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)
	Tue, Nov 18, 00:32

	Unknown Object (File)
	Mon, Nov 17, 12:04

	Unknown Object (File)
	Mon, Nov 17, 02:20

	Unknown Object (File)
	Sat, Nov 15, 23:50

	Unknown Object (File)
	Sat, Nov 15, 19:59

	Unknown Object (File)
	Sat, Nov 15, 19:36

	Unknown Object (File)
	Sat, Nov 15, 17:27

	Unknown Object (File)
	Sat, Nov 15, 07:38

View All 23 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