Fix rendering issue for LDAP configuration provider transactions
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F15534614: D7905.id17894.diff
	Thu, Apr 24, 1:03 AM

	F15527954: D7905.id17901.diff
	Tue, Apr 22, 7:50 AM

	F15524322: D7905.id.diff
	Mon, Apr 21, 7:26 AM

	F15521602: D7905.diff
	Sun, Apr 20, 3:24 PM

	F15493033: D7905.id17901.diff
	Sat, Apr 12, 8:57 PM

	F15478594: D7905.id17901.diff
	Apr 8 2025, 1:15 AM

	F15471418: D7905.diff
	Apr 5 2025, 4:54 AM

	F15429418: D7905.id17894.diff
	Mar 24 2025, 2:08 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