Summary | text_diff_renderer_inline calls itself, making subclassing impossible |
Queue | Horde Framework Packages |
Type | Bug |
State | Unconfirmed |
Priority | 1. Low |
Owners | |
Requester | mrten_horde (at) ii (dot) nl |
Created | 10/23/2018 (2395 days ago) |
Due | |
Updated | 10/23/2018 (2395 days ago) |
Assigned | |
Resolved | |
Milestone | |
Patch | Yes |
State ⇒ Unconfirmed
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ text_diff_renderer_inline calls itself, making subclassing impossible
Queue ⇒ Horde Framework Packages
Milestone ⇒
Patch ⇒ Yes
Horde_Text_Diff_Renderer_Inline you can subclass it. But on line ~160
the class makes a new instantiation of itself, thereby nullifying the
original subclass.
Small patch:
@@ -158,7 +158,8 @@
}
/* Get the diff in inline format. */
- $renderer = new Horde_Text_Diff_Renderer_inline
+ $class = static::class;
+ $renderer = new $class