6.0.0-beta1
7/6/25

[#2278] Delete Button
Summary Delete Button
Queue Horde Base
Queue Version HEAD
Type Enhancement
State Rejected
Priority 1. Low
Owners
Requester vijay.mahrra (at) es (dot) easynet (dot) net
Created 07/15/2005 (7296 days ago)
Due
Updated 11/08/2008 (6084 days ago)
Assigned
Resolved 11/08/2008 (6084 days ago)
Milestone
Patch No

History
11/08/2008 05:22:13 AM Chuck Hagenbuch State ⇒ Rejected
 
12/01/2005 11:31:27 PM Chuck Hagenbuch Comment #7
State ⇒ Stalled
Reply to this comment
Stalling until that happens. Will be happy to re-open later.
11/14/2005 04:28:10 PM vijay (dot) mahrra (at) es (dot) easynet (dot) net Comment #6 Reply to this comment
i have been away on a long holiday and have only just returned.  the 
patch was quite some time ago and so i will need to look at this once 
i've caught up with more pressing tasks at my job.
10/24/2005 02:19:19 PM Jan Schneider Comment #5 Reply to this comment
Are you going to update your patch?
07/18/2005 08:23:49 AM Jan Schneider Comment #4 Reply to this comment
I take your comment about this to mean that existing scripts need to
be converted from outputting html code in the script as well? They
should ideally be using Horde_Template?
Yes, and no. There is no reason to use Horde_Template, but all the 
HTML output, also the existing, should be moved to a template in 
templates/admin/datatree/.
07/18/2005 08:17:08 AM vijay (dot) mahrra (at) es (dot) easynet (dot) net Comment #3 Reply to this comment
Sorry, this was a bit of a sloppy effort.



However it is my hope that because the code does work you can see the 
benefit of this patch which does actually work.



I probably chose the directory instead of file in my CVS client to 
make the patch, hence the extra file, although the misnaming of the 
form was inexcusable.  As you can tell it was copied from the existing 
admin/*.php scripts and I was following the existing model used which 
output HTML too.



I take your comment about this to mean that existing scripts need to 
be converted from outputting html code in the script as well? They 
should ideally be using Horde_Template?
07/16/2005 12:43:36 PM Jan Schneider Comment #2
State ⇒ Feedback
Reply to this comment
This patch also contains files that don't belong to the change, you 
use form name that has nothing to do with the purpose, and please 
create a new template instead of outputting HTML code in the script.
07/15/2005 12:02:30 PM vijay (dot) mahrra (at) es (dot) easynet (dot) net Comment #1
Priority ⇒ 1. Low
Type ⇒ Enhancement
Summary ⇒ Delete Button
Queue ⇒ Horde Base
New Attachment: datatree.php.patch Download
State ⇒ New
Reply to this comment
This patch adds a 'Delete' button when you browse the datatree and 
click on an item which then removes the object (and any sub-items)

Saved Queries