Fix rendering issue for LDAP configuration provider transactions
ClosedPublic
Actions

Authored by epriestley on Jan 8 2014, 1:49 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Apr 15, 7:51 AM

	Unknown Object (File)
	Thu, Apr 11, 7:39 AM

	Unknown Object (File)
	Wed, Apr 10, 8:16 PM

	Unknown Object (File)
	Sat, Apr 6, 6:39 AM

	Unknown Object (File)
	Sat, Apr 6, 6:39 AM

	Unknown Object (File)
	Sat, Apr 6, 6:03 AM

	Unknown Object (File)
	Sat, Apr 6, 5:41 AM

	Unknown Object (File)
	Wed, Apr 3, 4:01 AM

View All Files

Subscribers

aran

Details

Reviewers

btrahan

Maniphest Tasks

T4290: PhabricatorAuthProviderConfigTransaction old value may contain other things than strings

Commits

Restricted Diffusion Commit
rP324ee4e4d539: Fix rendering issue for LDAP configuration provider transactions

Summary

Fixes T4290. At least one of the fields (realname) may have a list of items, and strlen(array('first', 'last')) produces the warning and stack trace in T4290.

Test Plan

Edited realname from an array value to an array value.
Hit error.
Applied patch.
No more error.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

btrahan accepted this revision.Jan 8 2014, 6:26 PM

Closed by commit rP324ee4e4d539 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

auth/

provider/

PhabricatorAuthProviderLDAP.php

2 lines

Diff 17901

View Options

src/applications/auth/provider/PhabricatorAuthProviderLDAP.php

Fix rendering issue for LDAP configuration provider transactionsClosedPublicActions