Page MenuHomePhabricator
Differential D21852

Fix repository URI generation for SVN commit identifiers represented with integer types
ClosedPublic
Actions

Authored by epriestley on Jun 3 2022, 10:00 PM.
Tags
None
Referenced Files
F15456621: D21852.diff
Sun, Mar 30, 10:50 AM
F15454161: D21852.diff
Sat, Mar 29, 4:40 PM
F15450962: D21852.id52083.diff
Fri, Mar 28, 7:19 PM
F15450944: D21852.id52082.diff
Fri, Mar 28, 7:13 PM
F15450935: D21852.id.diff
Fri, Mar 28, 7:11 PM
F15439089: D21852.diff
Wed, Mar 26, 5:42 AM
F15422835: D21852.id52083.diff
Sat, Mar 22, 11:01 AM
F15409334: D21852.diff
Wed, Mar 19, 4:15 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13588: PHP 8 Compatibility
Commits
rP2167016aebba: (stable) Fix repository URI generation for SVN commit identifiers represented…
rP1fc04e238f1d: Fix repository URI generation for SVN commit identifiers represented with…
Summary

Ref T13588. See PHI2198. Some pathways represent SVN commit identifiers (which are numeric) with integers; these raise an exception in phutil_nonempty_string(), which is strict.

(Ideally, commit identifiers wouldn't have mixed types, but that's a dramatically less practical change.)

Test Plan

Will deploy.

Diff Detail

Repository
rP Phabricator
Branch
scalar1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 25760
Build 35588: arc lint + arc unit

Revision Contents
Changeset List

PathSize
src/
applications/
repository/
storage/
2 lines

Diff 52082

View Options

src/applications/repository/storage/PhabricatorRepository.php

Loading...
Phabricator · Built by Phacility