Paths

Table of Contentst

AphrontFormTokenizerControl::readValueFromRequest
Needs ReviewPublic
Actions

Authored by avivey on Nov 24 2015, 12:02 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Feb 8, 11:52 PM

	Unknown Object (File)
	Sat, Feb 8, 11:52 PM

	Unknown Object (File)
	Mon, Jan 27, 4:33 PM

	Unknown Object (File)
	Mon, Jan 27, 6:49 AM

	Unknown Object (File)
	Sun, Jan 26, 7:00 PM

	Unknown Object (File)
	Fri, Jan 24, 10:04 AM

	Unknown Object (File)
	Fri, Jan 24, 10:04 AM

	Unknown Object (File)
	Jan 15 2025, 7:37 PM

View All Files

Subscribers

epriestley

Details

Reviewers

None

Group Reviewers

Blessed Reviewers

Summary

Not sure if deprecated or new... But it was failing when used in multi-page form.

Test Plan

Use $form->readFromRequest($request) with a tokenized field.

Diff Detail

Repository

rP Phabricator

Branch

master

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 9038
Build 10626: Run Core Tests
Build 10625: arc lint + arc unit

Event Timeline

avivey updated this revision to Diff 35229.Nov 24 2015, 12:02 AM

avivey retitled this revision from to AphrontFormTokenizerControl::readValueFromRequest.

avivey updated this object.

avivey edited the test plan for this revision. (Show Details)

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptNov 24 2015, 12:02 AM

Herald added a subscriber: epriestley. · View Herald Transcript

This isn't enough; When going "<< back", the value becomes string again.

Maybe T7029?

Might be close; I hit the form button for "back", and looked in darkconsole to see the value is a string. Stopped exploring at that point ATM.

Revision Contents
Changeset List

Path

Size

src/

view/

form/

control/

AphrontFormTokenizerControl.php

5 lines

Diff 35229

View Options

src/view/form/control/AphrontFormTokenizerControl.php

AphrontFormTokenizerControl::readValueFromRequestNeeds ReviewPublicActions