[Rt-commit] r17732 - in rt/3.999/trunk: .
sunnavy at bestpractical.com
sunnavy at bestpractical.com
Tue Jan 13 08:40:00 EST 2009
Author: sunnavy
Date: Tue Jan 13 08:40:00 2009
New Revision: 17732
Modified:
rt/3.999/trunk/ (props changed)
rt/3.999/trunk/lib/RT/Model/Attachment.pm
rt/3.999/trunk/lib/RT/Model/CustomField.pm
rt/3.999/trunk/lib/RT/Model/User.pm
Log:
r18761 at sunnavys-mb: sunnavy | 2009-01-13 21:39:43 +0800
references columns creator and last_updated_by to Model::User
Modified: rt/3.999/trunk/lib/RT/Model/Attachment.pm
==============================================================================
--- rt/3.999/trunk/lib/RT/Model/Attachment.pm (original)
+++ rt/3.999/trunk/lib/RT/Model/Attachment.pm Tue Jan 13 08:40:00 2009
@@ -94,12 +94,10 @@
column headers => type is 'blob', default is '';
};
-use Jifty::Plugin::ActorMetadata::Mixin::Model::ActorMetadata
- user_class => 'RT::Model::Principal',
- map => {
+use Jifty::Plugin::ActorMetadata::Mixin::Model::ActorMetadata map => {
created_by => 'creator',
created_on => 'created',
- };
+};
=head2 create
Modified: rt/3.999/trunk/lib/RT/Model/CustomField.pm
==============================================================================
--- rt/3.999/trunk/lib/RT/Model/CustomField.pm (original)
+++ rt/3.999/trunk/lib/RT/Model/CustomField.pm Tue Jan 13 08:40:00 2009
@@ -76,14 +76,12 @@
column disabled => max_length is 6, type is 'smallint', default is '0';
};
-use Jifty::Plugin::ActorMetadata::Mixin::Model::ActorMetadata
- user_class => 'RT::Model::Principal',
- map => {
+use Jifty::Plugin::ActorMetadata::Mixin::Model::ActorMetadata map => {
created_by => 'creator',
created_on => 'created',
updated_by => 'last_updated_by',
updated_on => 'last_updated'
- };
+};
our %FieldTypes = (
Modified: rt/3.999/trunk/lib/RT/Model/User.pm
==============================================================================
--- rt/3.999/trunk/lib/RT/Model/User.pm (original)
+++ rt/3.999/trunk/lib/RT/Model/User.pm Tue Jan 13 08:40:00 2009
@@ -129,14 +129,12 @@
use Jifty::Plugin::User::Mixin::Model::User; # name, email, email_confirmed
use Jifty::Plugin::Authentication::Password::Mixin::Model::User;
-use Jifty::Plugin::ActorMetadata::Mixin::Model::ActorMetadata
- user_class => 'RT::Model::Principal',
- map => {
+use Jifty::Plugin::ActorMetadata::Mixin::Model::ActorMetadata map => {
created_by => 'creator',
created_on => 'created',
updated_by => 'last_updated_by',
updated_on => 'last_updated'
- };
+};
# XXX TODO, merging params should 'just work' but does not
__PACKAGE__->column('email')->writable(1);
@@ -443,6 +441,7 @@
my ( $status, $user_msg ) = $self->SUPER::create(
# we need to feed creator and last_updated_by since current user doesn't have id yet
+# and principal id *should* be the same as user id
creator => $principal_id,
last_updated_by => $principal_id,
id => $principal_id,
More information about the Rt-commit
mailing list