Remove direct reads of form state from main Inline client code
ClosedPublic
Actions

Authored by epriestley on Thu, Mar 25, 8:56 PM.

Details

Reviewers: None
Maniphest Tasks: T13559: Include "Moved/Copied" gutter in 1-up diffs, and add aural cues, coverage, and fix inline counting
Commits: rP0f04d9e58458: Remove direct reads of form state from main Inline client code

Summary

Ref T13559. Instead of directly reading form state, make all callers use the "active" state instead. The state reads the form.

No functional changes, just clarifying responsiblites.

Test Plan

Created inlines, etc. See followup changes.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Thu, Mar 25, 8:56 PM

Harbormaster completed remote builds in B25324: Diff 51516.Thu, Mar 25, 8:57 PM

epriestley requested review of this revision.Thu, Mar 25, 8:57 PM

This revision was not accepted when it landed; it landed in state Needs Review.Mon, Mar 29, 4:00 PM

Closed by commit rP0f04d9e58458: Remove direct reads of form state from main Inline client code (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP0f04d9e58458: Remove direct reads of form state from main Inline client code.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

map.php

webroot/

rsrc/

js/

application/

diff/

DiffInline.js

28 lines

Diff 51537

View Options

resources/celerity/map.php