Changeset 236

Timestamp:

06/06/09 20:46:24 (15 years ago)

Author:

nanardon

Message:

• make massedit working

Location:

LATMOS-Accounts-Web

Files:

• lib/LATMOS/Accounts/Web/Controller/MassEdit.pm (modified) (2 diffs)
• lib/LATMOS/Accounts/Web/Model/AttrForms.pm (modified) (9 diffs)
• root/html/massedit/index.tt (modified) (2 diffs)

LATMOS-Accounts-Web/lib/LATMOS/Accounts/Web/Controller/MassEdit.pm

@@ -26 +26 @@
 
     my $base = $c->stash->{db} = $c->model('Accounts')->db;
+
     if ($c->req->param('otype') && $c->req->param('otype') ne $c->session->{filter}{otype}) {
         $c->session->{filter} = undef;
@@ -37 +38 @@
             $c->req->params->{attrcol} = $c->session->{massedit}{attrcol};
         }
+    }
+
+    if (($c->req->param('_change') || '') eq 'all') {
+        my %change;
+        foreach ($c->req->param('attrcol')) {
+            $change{$_} = $c->req->param($_) || undef;
+            delete($c->req->params->{$_});
+        }
+        foreach my $id ($c->req->param('objid')) {
+            my $obj = $base->get_object($c->req->param('otype'), $id) or next;
+            $obj->set_c_fields(%change);
+        }
+        $base->commit;
+    } elsif (($c->req->param('_change') || '') eq 'perobj') {
+        foreach my $id ($c->req->param('objid')) {
+            my $obj = $base->get_object($c->req->param('otype'), $id) or next;
+            my %change;
+            foreach ($c->req->param('attrcol')) {
+                $change{$_} = $c->req->param($id . '_' . $_) || undef;
+                delete($c->req->params->{$id . '_' . $_});
+            }
+            $obj->set_c_fields(%change);
+        }
+        $base->commit;
     }
     $c->forward('filter');

LATMOS-Accounts-Web/lib/LATMOS/Accounts/Web/Model/AttrForms.pm

@@ -19 +19 @@
     uidNumber => [ 'UID', 'text:6' ],
     gidNumber => [ 'GID', 'select:group:gidNumber' ],
-    manager => [ 'Responsable', 'select-N:user:uid' ],
-    managedBy => [ 'Responsable', 'select-N:user:uid' ],
+    manager => [ 'Responsable', 'select-N:user' ],
+    managedBy => [ 'Responsable', 'select-N:user' ],
     sn => [ 'Nom' ],
     givenName => [ 'Prénom' ],
@@ -37 +37 @@
     expire => [ 'Expire le', 'date' ],
     st => [ 'État (US)' ],
-    sutype => [ 'Structure admin.', 'select-N:sutype:name' ],
+    sutype => [ 'Structure admin.', 'select-N:sutype' ],
     exported => [ 'Exporté', 'checkbox' ],
     locked => [ 'Vérouillé', 'checkbox' ],
@@ -176 +176 @@
     $type ||= 'text';
     $type = 'text:6' if (($self->{form} || '') =~ /^group/ && $attr eq 'gidNumber');
+    my $htmlname = ($self->{object} ? $self->{object}->id . '_' : '') . $attr;
     for ($type) {
         /^textarea$/ and return sprintf(
             '<textarea name="%s" cols="40">%s</textarea>',
-            $self->escape($attr),
+            $self->escape($htmlname),
             $self->escape($self->attr_raw_value($attr)),
         );
@@ -195 +196 @@
             $html .= sprintf(
                 '<input type="text" name="%s" value="%s" size="12">',
-                $attr,
+                $htmlname,
                 $self->escape($date)
             );
@@ -201 +202 @@
             $html .= qq{
             <A HREF="#"
-                onClick="cal13.select(document.forms[0].$attr,'anchor13','dd/MM/yyyy');return false;"
-                TITLE="cal13.select(document.forms[0].$attr,'anchor13','dd/MM/yyyy'); return false;"
+                onClick="cal13.select(document.forms[0].$htmlname,'anchor13','dd/MM/yyyy');return false;"
+                TITLE="cal13.select(document.forms[0].$htmlname,'anchor13','dd/MM/yyyy'); return false;"
                 NAME="anchor13" ID="anchor13">
                 <img src="} . $self->{c}->uri_for(qw(/static icons view-calendar-day.png))
@@ -210 +211 @@
         /^checkbox$/ and do {
             return sprintf('<input type="checkbox" name="%s"%s>',
-                $attr,
+                $self->escape($htmlname),
                 $self->attr_raw_value($attr) ? '  checked="yes"' : ''
             );
@@ -219 +220 @@
             my $keyfield = $3;
             my $select = sprintf('<select name="%s">',
-                $self->escape($attr)) . "\n";
+                $self->escape($htmlname)) . "\n";
             $select .= '<option value="">--</option>' . "\n" if ($options =~ /N/);
             my $value = $self->attr_raw_value($attr) || '';
@@ -239 +240 @@
             return sprintf(
                 '<input type="text" name="%s" value="%s" size="%d">',
-                $attr,
+                $self->escape($htmlname),
                 $self->escape($self->attr_raw_value($attr)),
                 $1 || 30,
@@ -258 +259 @@
     my ($self) = @_;
     $self->{c}->req->param($self->label) or return;
+    my $prefix = $self->{object}->id . '_';
     $self->{object}->set_c_fields(
         map {
             $_ =>
                 ($attrs->{$_}[1] || '') eq 'checkbox'
-                ? ($self->{c}->req->param($_) ? 1 : 0)
-                : $self->{c}->req->param($_)
+                ? ($self->{c}->req->param("$prefix$_") ? 1 : 0)
+                : ($self->{c}->req->param("$prefix$_") || undef)
         } $self->attributes
     ) or return;

LATMOS-Accounts-Web/root/html/massedit/index.tt

@@ -47 +47 @@
 <form action="[% c.uri_for() %]" method="POST">
 <input type="hidden" name="otype" value="[% c.req.param('otype') %]">
+<input type="hidden" name="_change" value="all">
+[% FOREACH obj = objectslist %]
+<input type="hidden" name="objid" value="[% obj.id | html  %]">
+[% END %]
 [% FOREACH atcol = c.req.param('attrcol') %]
 [% NEXT IF NOT atcol %]
-<input type="hidden" name="attrcol" value="[% atcol %]">
 [% END %]
 <tr>
@@ -66 +69 @@
 <form action="[% c.uri_for() %]" method="POST">
 <input type="hidden" name="otype" value="[% c.req.param('otype') %]">
+<input type="hidden" name="_change" value="perobj">
 [% FOREACH atcol = c.req.param('attrcol') %]
 [% NEXT IF NOT atcol %]
-<input type="hidden" name="attrcol" value="[% atcol %]">
 [% END %]
 [% FOREACH obj = objectslist %]
 [% attrform = c.model('AttrForms', c.req.param('otype'), obj) %]
 <tr>
-<th>[% obj.id | html  %]</th>
+<th>[% obj.id | html  %]
+<input type="hidden" name="objid" value="[% obj.id | html  %]">
+</th>
 [% FOREACH atcol = c.req.param('attrcol') %]
 [% NEXT IF NOT atcol %]