Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the DBIx-Class-AuditLog CPAN distribution.

Report information
The Basics
Id: 85522
Status: new
Priority: 0/
Queue: DBIx-Class-AuditLog
People
Owner: Nobody in particular
Requestors: mst [...] shadowcat.co.uk
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Tue May 21 15:30:55 2013 mst [...] shadowcat.co.uk - Ticket created
Subject: Bogus relationship names
Date: Tue, 21 May 2013 19:30:40 +0000
To: bugs-DBIx-Class-AuditLog [...] rt.cpan.org
From: Matt S Trout <mst [...] shadowcat.co.uk>
The baseline schema uses AuditedTable, Field etc. as relationship names. Since relationship names map to perl methods, they should be of normal perl method naming - i.e. 'audited_table', 'field'. Currently your relationship names look like they're result source / class names, which they aren't, and this makes it extremely difficult to skim read AuditLog using code if you're familiar with normal DBIx::Class usage. Please can you duplicate the relationships under the correct names and deprecate the erroneous ones? -- Matt S Trout - Shadowcat Systems - Perl consulting with a commit bit and a clue http://shadowcat.co.uk/blog/matt-s-trout/ http://twitter.com/shadowcat_mst/ Email me now on mst (at) shadowcat.co.uk and let's chat about how our Catalyst commercial support, training and consultancy packages could help your team.