1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-01 11:12:42 +01:00
phorge-phorge/src/applications/harbormaster/artifact
epriestley b4faf2e63e Allow "harbormaster.createartifact" to decode raw HTTP parameter types of artifact properties
Summary:
Ref T11887. This isn't a great fix but makes the method behave properly until I get around to a real fix.

In the longer term, I want to convert all of this pluggable Harbormaster/Drydock stuff (blueprints, artifacts, build plans) to use EditEngine + EditField instead of the weird mishmash of older/custom stuff it currently uses. However, this is a more involved project to execute and I'd like to be in that area of the codebase first so it gets adequate testing.

Until that happens, just put a reasonble-ish mechanism in place to let artifacts correct inbound types. This is the only artifact type and only parameter which needs casting.

Test Plan:
  - Made a `curl` call to `harbormaster.createartifact` to create a URI artifact with `?...&ui.external=1`.
  - Before patch: type error on `ui.external` not being a boolean.
  - After patch: artifact created successfully.

Reviewers: chad

Reviewed By: chad

Subscribers: PHID-OPKG-gm6ozazyms6q6i22gyam

Maniphest Tasks: T11887

Differential Revision: https://secure.phabricator.com/D16887
2016-11-17 08:03:01 -08:00
..
HarbormasterArtifact.php Allow "harbormaster.createartifact" to decode raw HTTP parameter types of artifact properties 2016-11-17 08:03:01 -08:00
HarbormasterDrydockLeaseArtifact.php In Harbormaster, make sure artifacts are destroyed even if a build is aborted 2015-10-05 05:58:53 -07:00
HarbormasterFileArtifact.php Add harbormaster.createartifact 2015-08-15 07:28:56 -07:00
HarbormasterHostArtifact.php Allow Harbormaster to lease working copies from Drydock 2015-09-24 17:29:47 -07:00
HarbormasterURIArtifact.php Allow "harbormaster.createartifact" to decode raw HTTP parameter types of artifact properties 2016-11-17 08:03:01 -08:00
HarbormasterWorkingCopyArtifact.php Allow Harbormaster to lease working copies from Drydock 2015-09-24 17:29:47 -07:00