| Summary | monthdayyear object not logged in history |
| Queue | Horde Framework Packages |
| Type | Bug |
| State | Not Reproducible |
| Priority | 1. Low |
| Owners | |
| Requester | birnbacs (at) gmail (dot) com |
| Created | 10/08/2015 (3684 days ago) |
| Due | |
| Updated | 02/11/2016 (3558 days ago) |
| Assigned | 01/14/2016 (3586 days ago) |
| Resolved | 02/11/2016 (3558 days ago) |
| Github Issue Link | |
| Github Pull Request | |
| Milestone | |
| Patch | Yes |
State ⇒ Feedback
bug #14130the monthdayyearattribute is displayed both in the summary and the history.
Priority ⇒ 1. Low
Patch ⇒ Yes
Milestone ⇒
Queue ⇒ Horde Framework Packages
Due ⇒ 10/30/2015
Summary ⇒ monthdayyear object not logged in history
Type ⇒ Bug
State ⇒ Unconfirmed
ticket #14130.After fixing the monthdayyear object so that it would be usable in
whups I noticed that its history does not get displayed properly, the
amended value never showed up.
In pear module Form the class of the monthdayyear object is defined in
file Form/Type.php. Its init() method is defined in line 2851:
class Horde_Form_Type_monthdayyear extends Horde_Form_Type
{$start_year = '', $end_year = '', $picker = true,
$format_in = null, $format_out = '%x')
The init() method accepts 5 arguments, the last of which is
$format_out and has a default value of '%x'. In line 2865 the
protected variable $this->_format_out is unconditionally set to
$format_out. However, at least in the context of a user defined
attribute in whups, the init() method is called with $format_out =
null, which causes incorrect rendering of the variable saved in the
history (i.e. no rendering at all).
I fixed this by checking that $format_out is not null before
overwriting the protected variable:
if ($format_out != null) { // line 2865: don't overwrite the
default value with null
$this->_format_out = $format_out;