In-Portal Issue Tracker - In-Link
|
Viewing Issue Advanced Details |
|
ID:
|
Category:
|
Type:
|
Reproducibility:
|
Date Submitted:
|
Last Update:
|
910 |
[In-Portal CMS] Data Management |
bug report |
always |
2010-10-29 13:45 |
2011-03-30 09:58 |
|
Reporter:
|
Dmitry |
Platform:
|
|
|
Assigned To:
|
alex |
OS:
|
|
|
Priority:
|
normal |
OS Version:
|
|
|
Status:
|
closed |
Product Version:
|
5.1.1-B2 |
|
Product Build:
|
|
Resolution:
|
fixed |
|
|
ETA:
|
none |
Fixed in Version:
|
5.1.1-RC1 |
|
Reference:
|
http://groups.google.com/group/in-portal-bugs/browse_thread/thread/b0a6be017db86ef0 |
Change Log Message:
|
Fixes error when Modifying Pending Link on Front-end |
Estimate Points:
|
0 |
|
Summary:
|
0000910: Fatal error when Modifying Pending Link on Front-end |
Description:
|
Fatal error is shown when trying to modify your own link on a Front-end.
Happens ONLY when we are trying to MODIFY a PENDING link which NEVER has been approved yet. In other words it's MODIFY.PENDING of the originally submitted pending link.
*Fatal Error: *mail error: header To is required in
/home/sites/betainportal/web/5.1.1-beta2/full/core/kernel/utility/email_sen d.php
on line 1093 |
Steps To Reproduce:
|
|
Additional Information:
|
|
Relationships | related to | 0000704 | closed (5.1.0) | alex | In-Portal CMS | Improvements to "E-mail Template" editing | child of | 0000357 | closed (5.1.0) | alex | In-Portal CMS | Form modification detector (it's php part) works strange |
|
Attached Files:
|
link_modify_without_changing_it_gives_fatal_error.patch (1,993) 2010-10-31 08:21 http://tracker.in-portal.org/file_download.php?file_id=828&type=bug |
|
Issue History |
Date Modified |
Username |
Field |
Change |
2011-03-30 09:58 |
Dmitry |
Note Added: 0003388 |
|
2011-03-30 09:58 |
Dmitry |
Status |
resolved => closed |
2010-11-01 13:43 |
alex |
Changeset attached |
5.1.x r14024 |
2010-11-01 13:43 |
alex |
Note Added: 0002937 |
|
2010-11-01 13:43 |
alex |
Status |
reviewed and tested => resolved |
2010-11-01 13:43 |
alex |
Fixed in Version |
=> 5.1.1-RC1 |
2010-11-01 13:43 |
alex |
Resolution |
open => fixed |
2010-11-01 13:43 |
alex |
Assigned To |
!COMMUNITY => alex |
2010-11-01 10:08 |
Dmitry |
Note Added: 0002932 |
|
2010-11-01 10:08 |
Dmitry |
Status |
needs testing => reviewed and tested |
2010-10-31 08:27 |
alex |
Note Added: 0002930 |
|
2010-10-31 08:27 |
alex |
Assigned To |
Dmitry => !COMMUNITY |
2010-10-31 08:27 |
alex |
Developer |
=> alex |
2010-10-31 08:27 |
alex |
Status |
needs work => needs testing |
2010-10-31 08:21 |
alex |
File Added: link_modify_without_changing_it_gives_fatal_error.patch |
|
2010-10-31 08:20 |
alex |
Relationship added |
related to 0000704 |
2010-10-31 08:19 |
alex |
Relationship added |
child of 0000357 |
2010-10-29 13:46 |
Dmitry |
Assigned To |
=> Dmitry |
2010-10-29 13:46 |
Dmitry |
Status |
active => needs work |
2010-10-29 13:45 |
Dmitry |
New Issue |
|
2010-10-29 13:45 |
Dmitry |
Reference |
=> http://groups.google.com/group/in-portal-bugs/browse_thread/thread/b0a6be017db86ef0 |
2010-10-29 13:45 |
Dmitry |
Change Log Message |
=> Fixes error when Modifying Pending Link on Front-end |
Notes |
|
(0002930)
|
alex
|
2010-10-31 08:27
|
|
Synopsis:
Before In-Portal 5.1.0 release we have this behavior:
1. field ModifiedById of category items were always set, even after initial category item creation (admin or suggest form).
2. when sending email event and email was missing, then admin email was used.
After In-Portal 5.1.0 release behavior was changed to:
1. set ModifiedById field only, when some of the fields where changed
2. don't use admin's email as fallback email for all cases
What happened:
When user opened modify form of newly added link and pressed "Modify" button without changing any fields, then ModifiedById field was not changed, but attempt was made to get user's email (from that field), who changed that link to send notification to it. That all resulted to email email event recipient email.
Strange, but I thought, that we should inform owner of link about changes to it's record, not user, who actually changes it.
|
|
|
(0002932)
|
Dmitry
|
2010-11-01 10:08
|
|
Tested good, please commit. |
|
|
(0002937)
|
alex
|
2010-11-01 13:43
|
|
Fix committed to 5.1.x branch. Commit Message:
Fixes 0000910: Fatal error when Modifying Pending Link on Front-end |
|
|
(0003388)
|
Dmitry
|
2011-03-30 09:58
|
|
Closing, since 5.1.1 release has been released. |
|