[Rt-devel] RT 3.4.HEAD bug in ticket display?

Todd Chapman todd at chaka.net
Tue Jun 7 16:41:00 EDT 2005


On Mon, Jun 06, 2005 at 01:20:39PM -0400, Todd Chapman wrote:
> On Mon, Jun 06, 2005 at 01:11:33PM -0400, Jesse Vincent wrote:
> > 
> > 
> > 
> > On Mon, Jun 06, 2005 at 12:38:00PM -0400, Todd Chapman wrote:
> > > 
> > > The ShowSummary component uses this code to determine if
> > > custom fields need to be shown:
> > > 
> > > if ($Ticket->QueueObj->TicketCustomFields->First) {
> > > 
> > > The problem seems to be that TicketCustomFields does
> > > a check on right SeeQueue, so even if you have the
> > > right to SeeTicket and SeeCustomField, you can't see
> > > the custom fields for this ticket.
> > 
> > You can't see a queue's custom fields if you can't see the queue? That
> > sounds right, to me.
> 
> $Ticket->QueueObj->TicketCustomFields and $Ticket->CustomFields
> perform the same query except that the former does and SeeQueue
> check and the later does not.  If the policy is that to see
> a ticket's CF you need ShowTicket, SeeQueue, and SeeCustomField (whew!)
> then should not Ticket_Overlay.pm override Record::CustomFields
> with a call to Queue:TicketCustomFields ?

Any opinion?

> 
> 
> > 
> > 
> > > It seems that Queue::TicketCustomFields should be
> > > deprecated in favor of Record::CustomFields and the
> > 
> > That doesn't sound right. TicketCustomFields was added to differentiate
> > TicketCustomFields from TransactionCustomFields.
> 
> 
> _______________________________________________
> Rt-devel mailing list
> Rt-devel at lists.bestpractical.com
> http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-devel


More information about the Rt-devel mailing list