X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=SL%2FPresenter%2FPart.pm;h=3088eaa31c1f18db464c3af420252b6d44bf39f4;hb=58190585c35976c33c0594e920251ec8f7e3d40e;hp=4b7b15d3d1c244ac6dcf616de550e2848901de7b;hpb=686cf1ec3e3989b4f4b4b9a24aedee6ee2c3c660;p=kivitendo-erp.git
diff --git a/SL/Presenter/Part.pm b/SL/Presenter/Part.pm
index 4b7b15d3d..3088eaa31 100644
--- a/SL/Presenter/Part.pm
+++ b/SL/Presenter/Part.pm
@@ -5,19 +5,42 @@ use strict;
use SL::DB::Part;
use Exporter qw(import);
-our @EXPORT = qw(part_picker);
+our @EXPORT = qw(part_picker part);
+
+use Carp;
+
+sub part {
+ my ($self, $part, %params) = @_;
+
+ $params{display} ||= 'inline';
+
+ croak "Unknown display type '$params{display}'" unless $params{display} =~ m/^(?:inline|table-cell)$/;
+
+ my $text = join '', (
+ $params{no_link} ? '' : '',
+ $self->escape($part->partnumber),
+ $params{no_link} ? '' : '',
+ );
+ return $self->escaped_text($text);
+}
sub part_picker {
my ($self, $name, $value, %params) = @_;
$value = SL::DB::Manager::Part->find_by(id => $value) if $value && !ref $value;
my $id = delete($params{id}) || $self->name_to_id($name);
+ my $fat_set_item = delete $params{fat_set_item};
+
+ my @classes = $params{class} ? ($params{class}) : ();
+ push @classes, 'part_autocomplete';
+ push @classes, 'partpicker_fat_set_item' if $fat_set_item;
my $ret =
- $self->input_tag($name, (ref $value && $value->can('id') ? $value->id : ''), class => 'part_autocomplete', type => 'hidden', id => $id) .
- join('', map { $params{$_} ? $self->input_tag("", delete $params{$_}, id => "${id}_${_}", type => 'hidden') : '' } qw(column type unit convertible_unit)) .
- $self->input_tag("", (ref $value && $value->can('description')) ? $value->description : '', id => "${id}_name", %params);
+ $self->input_tag($name, (ref $value && $value->can('id') ? $value->id : ''), class => "@classes", type => 'hidden', id => $id) .
+ join('', map { $params{$_} ? $self->input_tag("", delete $params{$_}, id => "${id}_${_}", type => 'hidden') : '' } qw(type unit convertible_unit)) .
+ $self->input_tag("", ref $value ? $value->displayable_name : '', id => "${id}_name", %params);
+ $::request->layout->add_javascripts('autocomplete_part.js');
$::request->presenter->need_reinit_widgets($id);
$self->html_tag('span', $ret, class => 'part_picker');
@@ -31,11 +54,15 @@ __END__
=head1 NAME
-SL::Presenter::Part - Part lelated presenter stuff
+SL::Presenter::Part - Part related presenter stuff
=head1 SYNOPSIS
-see L
+ # Create an html link for editing/opening a part/service/assembly
+ my $object = my $object = SL::DB::Manager::Part->get_first;
+ my $html = SL::Presenter->get->part($object, display => 'inline');
+
+see also L
=head1 DESCRIPTION
@@ -43,8 +70,29 @@ see L
=head1 FUNCTIONS
+=over 2
+
+=item C
+
+Returns a rendered version (actually an instance of
+L) of the part object C<$object>
+
+C<%params> can include:
+
=over 4
+=item * display
+
+Either C (the default) or C. At the moment both
+representations are identical and produce the part's name linked
+to the corresponding 'edit' action.
+
+=back
+
+=back
+
+=over 2
+
=item C
All-in-one picker widget for parts. The name will be both id and name
@@ -52,7 +100,7 @@ of the resulting hidden C input field (but the ID can be
overwritten with C<$params{id}>).
An additional dummy input will be generated which is used to find
-parts. For a detailed description of it's behaviour, see section
+parts. For a detailed description of its behaviour, see section
C.
C<$value> can be a parts id or a C instance.
@@ -68,12 +116,15 @@ C.
If C<%params> contains C only parts with a unit
that's convertible to unit will be used for autocompletion.
-Obsolete parts will by default not displayed for selection. However they are
+Obsolete parts will by default not be displayed for selection. However they are
accepted as default values and can persist during updates. As with other
selectors though, they are not selectable once overridden.
-Currently you must include C in your controller, the
-presenter can not do this from the template.
+C will register it's javascript for inclusion in the next header
+rendering. If you write a standard controller that only call C once, it
+will just work. In case the header is generated in a different render call
+(multiple blocks, ajax, old C style controllers) you need to
+include C yourself.
=back
@@ -116,7 +167,7 @@ Should not require a feedback/check loop in the common case
=item *
-Should not be constraint to exact matches
+Should not be constrained to exact matches
=back
@@ -145,7 +196,7 @@ ambiguity.
=item 5
-An internal status of the part picker, indicating wether id input and dummy
+An internal status of the part picker, indicating whether id input and dummy
input are consistent. After leaving the dummy input the part picker must
place itself in a consistent status.