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

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

Tags

None

Referenced Files

	F15193431: D21647.id51516.diff
	Sat, Feb 22, 5:22 AM

	F15193200: D21647.id51537.diff
	Sat, Feb 22, 4:23 AM

	Unknown Object (File)
	Fri, Feb 21, 12:21 PM

	Unknown Object (File)
	Wed, Feb 19, 7:06 PM

	Unknown Object (File)
	Mon, Feb 17, 6:13 PM

	Unknown Object (File)
	Sat, Feb 15, 3:29 PM

	Unknown Object (File)
	Sat, Feb 15, 12:54 PM

	Unknown Object (File)
	Thu, Feb 13, 2:30 AM

View All Files

Subscribers

None

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.Mar 25 2021, 8:56 PM

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

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

This revision was not accepted when it landed; it landed in state Needs Review.Mar 29 2021, 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