Fix rendering issue for LDAP configuration provider transactions
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F19153049: D7905.id.diff
	Thu, Dec 11, 7:06 AM

	F19147191: D7905.diff
	Wed, Dec 10, 4:07 PM

	F18877339: D7905.id.diff
	Nov 6 2025, 7:34 AM

	F18874967: D7905.diff
	Nov 5 2025, 4:25 PM

	F18828275: D7905.id17901.diff
	Oct 24 2025, 3:34 PM

	F18741000: D7905.id.diff
	Oct 2 2025, 7:25 AM

	F18739098: D7905.diff
	Oct 1 2025, 7:44 PM

	F18617919: D7905.diff
	Sep 14 2025, 10:35 PM

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