|
In-Portal Issue Tracker - In-Portal CMS
|
|
Viewing Issue Advanced Details |
|
|
ID:
|
Category:
|
Type:
|
Reproducibility:
|
Date Submitted:
|
Last Update:
|
|
1275 |
[In-Portal CMS] Data Management |
bug report |
always |
2012-05-15 14:09 |
2012-07-25 05:30 |
|
|
Reporter:
|
Dmitry |
Platform:
|
|
|
|
Assigned To:
|
alex |
OS:
|
|
|
|
Priority:
|
normal |
OS Version:
|
|
|
|
Status:
|
closed |
Product Version:
|
5.2.0-B1 |
|
|
Product Build:
|
|
Resolution:
|
fixed |
|
| |
|
ETA:
|
none |
Fixed in Version:
|
5.2.0-RC1 |
|
|
Reference:
|
https://groups.google.com/d/topic/in-portal-bugs/QV-1VDJp11U/discussion |
|
Change Log Message:
|
Fixed user can't subscribe to newsletter |
|
Estimate Points:
|
0 |
|
|
Summary:
|
0001275: User can't Subscribe to Newsletter |
|
Description:
|
Can't Subscribe to Newsletter due to code issues in u_OnSubscribeUser event:
Line 759 $this->RemoveRequiredFields($object);
Line 790 if ( $object->isRequired('Username') ) {
$object->SetDBField('Username', $user_email);
}
First mentioned line ensures that 'Username' field is never set, and,
as result - subscriber-user record is not created with error "username
is too short" (as fact, it is empty, so it length is 0). |
|
Steps To Reproduce:
|
|
|
Additional Information:
|
To fix we need to check $object->isRequired('Username') before we do $this->RemoveRequiredFields($object); |
| Relationships | | parent of | 0001107 | closed (5.2.0) | alex | Logical changes on user registration/my profile/forgot password forms |
|
|
Attached Files:
|
 User-cant-Subscribe-to-Newsletter.patch (658) 2012-05-15 14:09
http://tracker.in-portal.org/file_download.php?file_id=1649&type=bug
User-cant-Subscribe-to-Newsletter_v2.patch (829) 2012-05-29 08:57
http://tracker.in-portal.org/file_download.php?file_id=1684&type=bug |
|
|
Issue History |
|
Date Modified |
Username |
Field |
Change |
|
2012-07-25 05:30 |
alex |
Note Added: 0004881 |
|
|
2012-07-25 05:30 |
alex |
Status |
resolved => closed |
|
2012-05-29 08:58 |
alex |
Changeset attached |
5.2.x r15370 |
|
2012-05-29 08:58 |
alex |
Note Added: 0004683 |
|
|
2012-05-29 08:58 |
alex |
Status |
reviewed and tested => resolved |
|
2012-05-29 08:58 |
alex |
Fixed in Version |
=> 5.2.0-RC1 |
|
2012-05-29 08:58 |
alex |
Resolution |
open => fixed |
|
2012-05-29 08:58 |
alex |
Assigned To |
!COMMUNITY => alex |
|
2012-05-29 08:57 |
alex |
File Added: User-cant-Subscribe-to-Newsletter_v2.patch |
|
|
2012-05-29 08:57 |
alex |
Note Added: 0004682 |
|
|
2012-05-29 08:57 |
alex |
Status |
needs testing => reviewed and tested |
|
2012-05-16 08:36 |
Dmitry |
Assigned To |
=> !COMMUNITY |
|
2012-05-16 08:36 |
Dmitry |
Developer |
=> Dmitry |
|
2012-05-16 08:36 |
Dmitry |
Status |
active => needs testing |
|
2012-05-15 14:12 |
Dmitry |
Summary |
User can't Subscribe to a Newsletter => User can't Subscribe to Newsletter |
|
2012-05-15 14:12 |
Dmitry |
Relationship added |
parent of 0001107 |
|
2012-05-15 14:09 |
Dmitry |
New Issue |
|
|
2012-05-15 14:09 |
Dmitry |
File Added: User-cant-Subscribe-to-Newsletter.patch |
|
|
2012-05-15 14:09 |
Dmitry |
Reference |
=> https://groups.google.com/d/topic/in-portal-bugs/QV-1VDJp11U/discussion |
|
2012-05-15 14:09 |
Dmitry |
Change Log Message |
=> Fixed user can't subscribe to newsletter |
|
2012-05-15 14:09 |
Dmitry |
Estimate Points |
=> 0 |