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

[Todd Chapman]

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 ?


> 
> 
> > 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.