Opened 14 years ago
Closed 14 years ago
#2515 closed enhancement (fixed)
[patch] LDAP username compability
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 1.2.6 | Priority: | trivial |
| Severity: | Version: | ||
| Component: | Activity | Keywords: | has-patch |
| Cc: |
Description
LDAP accounts have username like: name.surname[at]domain.tdl
To support that on @mentions feature, can the username pattern change, by adding a dot as an allowed char?
From: /[@]+([A-Za-z0-9-_]+)/
To: /[@]+([A-Za-z0-9-_\.]+)/
Files: /trunk @rev 2722
- bp-activity-filters.php (line 96)
- bp-activity-notifications.php (line 7)
Is that a problem?
Attachments (1)
Change History (10)
#2
@
14 years ago
- Keywords has-patch added; ldap mention removed
- Owner set to sorich87
- Status changed from new to accepted
#3
@
14 years ago
- Summary changed from LDAP username compability to [patch] LDAP username compability
#5
@
14 years ago
Don't currently have an LDAP installation to test this on. If anyone does, and can confirm for me that this fixes their issue, we'll get it in for 1.2.6.
#7
in reply to:
↑ 6
;
follow-up:
↓ 8
@
14 years ago
Replying to heliocorreia:
johnjamesjacoby, it worked for me.
heliocorreia, You mean the patch works or that you didn't even experience a problem to begin with?
#8
in reply to:
↑ 7
@
14 years ago
Replying to paulhastings0:
heliocorreia, You mean the patch works or that you didn't even experience a problem to begin with?
paulhastings0, the patch works with my LDAP/BuddyPress enviroment.
Nothing really to test with this change...