use SL::DBUtils;
use SL::Helper::Flash;
use SL::Locale::String;
+use SL::Util qw(trim);
+use SL::VATIDNr;
+use SL::Webdav;
+use SL::ZUGFeRD;
use SL::Controller::Helper::GetModels;
use SL::Controller::Helper::ReportGenerator;
use SL::Controller::Helper::ParseFilter;
use SL::DB::Customer;
use SL::DB::Vendor;
use SL::DB::Business;
+use SL::DB::ContactDepartment;
+use SL::DB::ContactTitle;
use SL::DB::Employee;
+use SL::DB::Greeting;
use SL::DB::Language;
use SL::DB::TaxZone;
use SL::DB::Note;
use SL::DB::PaymentTerm;
use SL::DB::Pricegroup;
+use SL::DB::Price;
use SL::DB::Contact;
use SL::DB::FollowUp;
use SL::DB::FollowUpLink;
use Data::Dumper;
use Rose::Object::MakeMethods::Generic (
- 'scalar --get_set_init' => [ qw(customer_models vendor_models) ],
+ scalar => [ qw(user_has_edit_rights) ],
+ 'scalar --get_set_init' => [ qw(customer_models vendor_models zugferd_settings) ],
);
# safety
-__PACKAGE__->run_before(
- sub {
- $::auth->assert('customer_vendor_edit');
- },
- except => [ qw(ajaj_autocomplete) ],
-);
__PACKAGE__->run_before(
'_instantiate_args',
only => [
'delete',
'delete_contact',
'delete_shipto',
+ 'delete_additional_billing_address',
]
);
'show',
'update',
'ajaj_get_shipto',
+ 'ajaj_get_additional_billing_address',
'ajaj_get_contact',
+ 'ajax_list_prices',
]
);
# make sure this comes after _load_customer_vendor
-__PACKAGE__->run_before(
- '_check_customer_vendor_all_edit',
- only => [
- 'edit',
- 'show',
- 'update',
- 'delete',
- 'save',
- 'save_and_ap_transaction',
- 'save_and_ar_transaction',
- 'save_and_close',
- 'save_and_invoice',
- 'save_and_order',
- 'save_and_quotation',
- 'save_and_rfq',
- 'delete',
- 'delete_contact',
- 'delete_shipto',
- ]
-);
+__PACKAGE__->run_before('_check_auth');
__PACKAGE__->run_before(
'_create_customer_vendor',
__PACKAGE__->run_before('normalize_name');
+my @ADDITIONAL_BILLING_ADDRESS_COLUMNS = qw(name department_1 department_2 contact street zipcode city country gln email phone fax default_address);
sub action_add {
my ($self) = @_;
$self->_pre_render();
- $self->{cv}->assign_attributes(hourly_rate => $::instance_conf->get_customer_hourly_rate) if $self->{cv}->is_customer;
+
+ if ($self->{cv}->is_customer) {
+ $self->{cv}->assign_attributes(hourly_rate => $::instance_conf->get_customer_hourly_rate);
+ $self->{cv}->salesman_id(SL::DB::Manager::Employee->current->id) if !$::auth->assert('customer_vendor_all_edit', 1);
+ }
$self->render(
'customer_vendor/form',
}
}
+sub _check_ustid_taxnumber_unique {
+ my ($self) = @_;
+
+ my %cfg;
+ if ($self->is_vendor()) {
+ %cfg = (should_check => $::instance_conf->get_vendor_ustid_taxnummer_unique,
+ manager_class => 'SL::DB::Manager::Vendor',
+ err_ustid => t8('A vendor with the same VAT ID already exists.'),
+ err_taxnumber => t8('A vendor with the same taxnumber already exists.'),
+ );
+
+ } elsif ($self->is_customer()) {
+ %cfg = (should_check => $::instance_conf->get_customer_ustid_taxnummer_unique,
+ manager_class => 'SL::DB::Manager::Customer',
+ err_ustid => t8('A customer with the same VAT ID already exists.'),
+ err_taxnumber => t8('A customer with the same taxnumber already exists.'),
+ );
+
+ } else {
+ return;
+ }
+
+ my @errors;
+
+ if ($cfg{should_check}) {
+ my $do_clean_taxnumber = sub { my $n = $_[0]; $n //= ''; $n =~ s{[[:space:].-]+}{}g; return $n};
+
+ my $clean_ustid = SL::VATIDNr->clean($self->{cv}->ustid);
+ my $clean_taxnumber = $do_clean_taxnumber->($self->{cv}->taxnumber);
+
+ if (!($clean_ustid || $clean_taxnumber)) {
+ return t8('VAT ID and/or taxnumber must be given.');
+
+ } else {
+ my $clean_number = $clean_ustid;
+ if ($clean_number) {
+ my $entries = $cfg{manager_class}->get_all(query => ['!id' => $self->{cv}->id, '!ustid' => undef, '!ustid' => ''], select => ['ustid'], distinct => 1);
+ if (any { $clean_number eq SL::VATIDNr->clean($_->ustid) } @$entries) {
+ push @errors, $cfg{err_ustid};
+ }
+ }
+
+ $clean_number = $clean_taxnumber;
+ if ($clean_number) {
+ my $entries = $cfg{manager_class}->get_all(query => ['!id' => $self->{cv}->id, '!taxnumber' => undef, '!taxnumber' => ''], select => ['taxnumber'], distinct => 1);
+ if (any { $clean_number eq $do_clean_taxnumber->($_->taxnumber) } @$entries) {
+ push @errors, $cfg{err_taxnumber};
+ }
+ }
+ }
+ }
+
+ return join "\n", @errors if @errors;
+ return;
+}
+
sub _save {
my ($self) = @_;
$::dispatcher->end_request;
}
+ $self->{cv}->greeting(trim $self->{cv}->greeting);
+ my $save_greeting = $self->{cv}->greeting
+ && $::instance_conf->get_vc_greetings_use_textfield
+ && SL::DB::Manager::Greeting->get_all_count(where => [description => $self->{cv}->greeting]) == 0;
+
+ $self->{contact}->cp_title(trim($self->{contact}->cp_title));
+ my $save_contact_title = $self->{contact}->cp_title
+ && $::instance_conf->get_contact_titles_use_textfield
+ && SL::DB::Manager::ContactTitle->get_all_count(where => [description => $self->{contact}->cp_title]) == 0;
+
+ $self->{contact}->cp_abteilung(trim($self->{contact}->cp_abteilung));
+ my $save_contact_department = $self->{contact}->cp_abteilung
+ && $::instance_conf->get_contact_departments_use_textfield
+ && SL::DB::Manager::ContactDepartment->get_all_count(where => [description => $self->{contact}->cp_abteilung]) == 0;
+
my $db = $self->{cv}->db;
$db->with_transaction(sub {
}
}
+ my $ustid_taxnumber_error = $self->_check_ustid_taxnumber_unique;
+ $::form->error($ustid_taxnumber_error) if $ustid_taxnumber_error;
+
$self->{cv}->save(cascade => 1);
+ SL::DB::Greeting->new(description => $self->{cv}->greeting)->save if $save_greeting;
+
$self->{contact}->cp_cv_id($self->{cv}->id);
if( $self->{contact}->cp_name ne '' || $self->{contact}->cp_givenname ne '' ) {
+ SL::DB::ContactTitle ->new(description => $self->{contact}->cp_title) ->save if $save_contact_title;
+ SL::DB::ContactDepartment->new(description => $self->{contact}->cp_abteilung)->save if $save_contact_department;
+
$self->{contact}->save(cascade => 1);
}
$self->{shipto}->save(cascade => 1);
}
+ if ($self->is_customer && any { $self->{additional_billing_address}->$_ ne '' } @ADDITIONAL_BILLING_ADDRESS_COLUMNS) {
+ $self->{additional_billing_address}->customer_id($self->{cv}->id);
+ $self->{additional_billing_address}->save(cascade => 1);
+
+ # Make sure only one address per customer has "default address" set.
+ if ($self->{additional_billing_address}->default_address) {
+ SL::DB::Manager::AdditionalBillingAddress->update_all(
+ set => { default_address => 0, },
+ where => [
+ customer_id => $self->{cv}->id,
+ '!id' => $self->{additional_billing_address}->id,
+ ]);
+ }
+ }
+
my $snumbers = $self->is_vendor() ? 'vendornumber_'. $self->{cv}->vendornumber : 'customernumber_'. $self->{cv}->customernumber;
SL::DB::History->new(
trans_id => $self->{cv}->id,
push(@redirect_params, shipto_id => $self->{shipto}->shipto_id);
}
+ if ( $self->is_customer && $self->{additional_billing_address}->id ) {
+ push(@redirect_params, additional_billing_address_id => $self->{additional_billing_address}->id);
+ }
+
$self->redirect_to(@redirect_params);
}
$self->action_edit();
}
+sub action_delete_additional_billing_address {
+ my ($self) = @_;
+
+ my $db = $self->{additional_billing_address}->db;
+
+ if ( !$self->{additional_billing_address}->id ) {
+ SL::Helper::Flash::flash('error', $::locale->text('No address selected to delete'));
+ } else {
+ $db->with_transaction(sub {
+ if ( $self->{additional_billing_address}->used ) {
+ $self->{additional_billing_address}->detach;
+ $self->{additional_billing_address}->save(cascade => 1);
+ SL::Helper::Flash::flash('info', $::locale->text('Address is in use and was flagged invalid.'));
+ } else {
+ $self->{additional_billing_address}->delete(cascade => 1);
+ SL::Helper::Flash::flash('info', $::locale->text('Address deleted.'));
+ }
+
+ 1;
+ }) || die($db->error);
+
+ $self->{additional_billing_address} = SL::DB::AdditionalBillingAddress->new;
+ }
+
+ $self->action_edit;
+}
sub action_search {
my ($self) = @_;
sub action_get_delivery {
my ($self) = @_;
- $::auth->assert('sales_all_edit');
+ $::auth->assert('sales_all_edit') if $self->is_customer();
+ $::auth->assert('purchase_all_edit') if $self->is_vendor();
my $dbh = $::form->get_standard_dbh();
$self->render(\SL::JSON::to_json($data), { type => 'json', process => 0 });
}
+sub action_ajaj_get_additional_billing_address {
+ my ($self) = @_;
+
+ my $data = {
+ additional_billing_address => {
+ map { ($_ => $self->{additional_billing_address}->$_) } ('id', @ADDITIONAL_BILLING_ADDRESS_COLUMNS)
+ },
+ };
+
+ $self->render(\SL::JSON::to_json($data), { type => 'json', process => 0 });
+}
+
sub action_ajaj_get_contact {
my ($self) = @_;
}
# if someone types something, and hits enter, assume he entered the full name.
- # if something matches, treat that as sole match
- # unfortunately get_models can't do more than one per package atm, so we d it
+ # if something matches, treat that as the sole match
+ # unfortunately get_models can't do more than one per package atm, so we do it
# the oldfashioned way.
if ($::form->{prefer_exact}) {
my $exact_matches;
if (1 == scalar @{ $exact_matches = $manager->get_all(
query => [
obsolete => 0,
- (salesman_id => SL::DB::Manager::Employee->current->id) x !$::auth->assert('customer_vendor_all_edit', 1),
or => [
name => { ilike => $::form->{filter}{'all:substr:multi::ilike'} },
$number => { ilike => $::form->{filter}{'all:substr:multi::ilike'} },
$_[0]->render('customer_vendor/test_page');
}
+sub action_ajax_list_prices {
+ my ($self, %params) = @_;
+
+ my $report = SL::ReportGenerator->new(\%::myconfig, $::form);
+ my @columns = qw(partnumber description price);
+ my @visible = qw(partnumber description price);
+ my @sortable = qw(partnumber description price);
+
+ my %column_defs = (
+ partnumber => { text => $::locale->text('Part Number'), sub => sub { $_[0]->parts->partnumber } },
+ description => { text => $::locale->text('Part Description'), sub => sub { $_[0]->parts->description } },
+ price => { text => $::locale->text('Price'), sub => sub { $::form->format_amount(\%::myconfig, $_[0]->price, 2) }, align => 'right' },
+ );
+
+ $::form->{sort_by} ||= 'partnumber';
+ $::form->{sort_dir} //= 1;
+
+ for my $col (@sortable) {
+ $column_defs{$col}{link} = $self->url_for(
+ action => 'ajax_list_prices',
+ callback => $::form->{callback},
+ db => $::form->{db},
+ id => $self->{cv}->id,
+ sort_by => $col,
+ sort_dir => ($::form->{sort_by} eq $col ? 1 - $::form->{sort_dir} : $::form->{sort_dir})
+ );
+ }
+
+ map { $column_defs{$_}{visible} = 1 } @visible;
+
+ my $pricegroup;
+ $pricegroup = $self->{cv}->pricegroup->pricegroup if $self->{cv}->pricegroup;
+
+ $report->set_columns(%column_defs);
+ $report->set_column_order(@columns);
+ $report->set_options(allow_pdf_export => 0, allow_csv_export => 0);
+ $report->set_sort_indicator($::form->{sort_by}, $::form->{sort_dir});
+ $report->set_export_options(@{ $params{report_generator_export_options} || [] });
+ $report->set_options(
+ %{ $params{report_generator_options} || {} },
+ output_format => 'HTML',
+ top_info_text => $::locale->text('Pricegroup') . ': ' . $pricegroup,
+ title => $::locale->text('Price List'),
+ );
+
+ my $sort_param = $::form->{sort_by} eq 'price' ? 'price' :
+ $::form->{sort_by} eq 'description' ? 'parts.description' :
+ 'parts.partnumber';
+ $sort_param .= ' ' . ($::form->{sort_dir} ? 'ASC' : 'DESC');
+ my $prices = SL::DB::Manager::Price->get_all(where => [ pricegroup_id => $self->{cv}->pricegroup_id ],
+ sort_by => $sort_param,
+ with_objects => 'parts');
+
+ $self->report_generator_list_objects(report => $report, objects => $prices, layout => 0, header => 0);
+}
+
sub is_vendor {
return $::form->{db} eq 'vendor';
}
$self->{shipto}->assign_attributes(%{$::form->{shipto}});
$self->{shipto}->module('CT');
+ if ($self->is_customer) {
+ if ( $::form->{additional_billing_address}->{id} ) {
+ $self->{additional_billing_address} = SL::DB::AdditionalBillingAddress->new(id => $::form->{additional_billing_address}->{id})->load;
+ } else {
+ $self->{additional_billing_address} = SL::DB::AdditionalBillingAddress->new;
+ }
+ $self->{additional_billing_address}->assign_attributes(%{ $::form->{additional_billing_address} });
+ }
+
if ( $::form->{contact}->{cp_id} ) {
$self->{contact} = SL::DB::Contact->new(cp_id => $::form->{contact}->{cp_id})->load();
} else {
$self->{shipto} = SL::DB::Shipto->new();
}
+ if ($self->is_customer) {
+ if ( $::form->{additional_billing_address_id} ) {
+ $self->{additional_billing_address} = SL::DB::AdditionalBillingAddress->new(id => $::form->{additional_billing_address_id})->load;
+ die($::locale->text('Error')) if $self->{additional_billing_address}->customer_id != $self->{cv}->id;
+
+ } else {
+ $self->{additional_billing_address} = SL::DB::AdditionalBillingAddress->new;
+ }
+ }
+
if ( $::form->{contact_id} ) {
$self->{contact} = SL::DB::Contact->new(cp_id => $::form->{contact_id})->load();
}
}
-sub _check_customer_vendor_all_edit {
- my ($self) = @_;
+sub _may_access_action {
+ my ($self, $action) = @_;
- unless ($::auth->assert('customer_vendor_all_edit', 1)) {
- die($::locale->text("You don't have the rights to edit this customer.") . "\n")
- if $self->{cv}->is_customer and
- SL::DB::Manager::Employee->current->id != $self->{cv}->salesman_id;
- };
-};
+ my $is_new = !$self->{cv} || !$self->{cv}->id;
+ my $is_own_customer = !$is_new
+ && $self->{cv}->is_customer
+ && (SL::DB::Manager::Employee->current->id == $self->{cv}->salesman_id);
+ my $has_edit_rights = $::auth->assert('customer_vendor_all_edit', 1);
+ $has_edit_rights ||= $::auth->assert('customer_vendor_edit', 1) && ($is_new || $is_own_customer);
+ my $needs_edit_rights = $action =~ m{^(?:add|save|delete|update)};
+
+ $self->user_has_edit_rights($has_edit_rights);
+
+ return 1 if $has_edit_rights;
+ return 0 if $needs_edit_rights;
+ return 1;
+}
+
+sub _check_auth {
+ my ($self, $action) = @_;
+
+ if (!$self->_may_access_action($action)) {
+ $::auth->deny_access;
+ }
+}
sub _create_customer_vendor {
my ($self) = @_;
$self->{note_followup} = SL::DB::FollowUp->new();
$self->{shipto} = SL::DB::Shipto->new();
+ $self->{additional_billing_address} = SL::DB::AdditionalBillingAddress->new if $self->is_customer;
$self->{contact} = $self->_new_contact_object;
}
$self->{all_employees} = SL::DB::Manager::Employee->get_all(query => [ deleted => 0 ]);
- $query =
- 'SELECT DISTINCT(greeting)
- FROM customer
- WHERE greeting IS NOT NULL AND greeting != \'\'
- UNION
- SELECT DISTINCT(greeting)
- FROM vendor
- WHERE greeting IS NOT NULL AND greeting != \'\'
- ORDER BY greeting';
- $self->{all_greetings} = [
- map(
- { $_->{greeting}; }
- selectall_hashref_query($::form, $dbh, $query)
- )
- ];
-
- $query =
- 'SELECT DISTINCT(cp_title) AS title
- FROM contacts
- WHERE cp_title IS NOT NULL AND cp_title != \'\'
- ORDER BY cp_title';
- $self->{all_titles} = [
- map(
- { $_->{title}; }
- selectall_hashref_query($::form, $dbh, $query)
- )
- ];
+ $self->{all_greetings} = SL::DB::Manager::Greeting->get_all_sorted();
+ if ($self->{cv}->id && $self->{cv}->greeting && !grep {$self->{cv}->greeting eq $_->description} @{$self->{all_greetings}}) {
+ unshift @{$self->{all_greetings}}, (SL::DB::Greeting->new(description => $self->{cv}->greeting));
+ }
+
+ $self->{all_contact_titles} = SL::DB::Manager::ContactTitle->get_all_sorted();
+ foreach my $contact (@{ $self->{cv}->contacts }) {
+ if ($contact->cp_title && !grep {$contact->cp_title eq $_->description} @{$self->{all_contact_titles}}) {
+ unshift @{$self->{all_contact_titles}}, (SL::DB::ContactTitle->new(description => $contact->cp_title));
+ }
+ }
+
+ $self->{all_contact_departments} = SL::DB::Manager::ContactDepartment->get_all_sorted();
+ foreach my $contact (@{ $self->{cv}->contacts }) {
+ if ($contact->cp_abteilung && !grep {$contact->cp_abteilung eq $_->description} @{$self->{all_contact_departments}}) {
+ unshift @{$self->{all_contact_departments}}, (SL::DB::ContactDepartment->new(description => $contact->cp_abteilung));
+ }
+ }
$self->{all_currencies} = SL::DB::Manager::Currency->get_all();
$self->{all_pricegroups} = SL::DB::Manager::Pricegroup->get_all_sorted(query => [ or => [ id => $self->{cv}->pricegroup_id, obsolete => 0 ] ]);
}
- $query =
- 'SELECT DISTINCT(cp_abteilung) AS department
- FROM contacts
- WHERE cp_abteilung IS NOT NULL AND cp_abteilung != \'\'
- ORDER BY cp_abteilung';
- $self->{all_departments} = [
- map(
- { $_->{department}; }
- selectall_hashref_query($::form, $dbh, $query)
- )
- ];
-
$self->{contacts} = $self->{cv}->contacts;
$self->{contacts} ||= [];
$self->{shiptos} = $self->{cv}->shipto;
$self->{shiptos} ||= [];
+ if ($self->is_customer) {
+ $self->{additional_billing_addresses} = $self->{cv}->additional_billing_addresses;
+ $self->{additional_billing_addresses} ||= [];
+ }
+
$self->{notes} = SL::DB::Manager::Note->get_all(
query => [
trans_id => $self->{cv}->id,
],
);
}
+
+ if ($self->{cv}->number && $::instance_conf->get_webdav) {
+ my $webdav = SL::Webdav->new(
+ type => $self->is_customer ? 'customer'
+ : $self->is_vendor ? 'vendor'
+ : undef,
+ number => $self->{cv}->number,
+ );
+ my @all_objects = $webdav->get_all_objects;
+ @{ $self->{template_args}->{WEBDAV} } = map { { name => $_->filename,
+ type => t8('File'),
+ link => File::Spec->catfile($_->full_filedescriptor),
+ } } @all_objects;
+ }
+
$self->{template_args} ||= {};
$::request->{layout}->add_javascripts('kivi.CustomerVendor.js');
sub _setup_form_action_bar {
my ($self) = @_;
+ my $no_rights = $self->user_has_edit_rights ? undef
+ : $self->{cv}->is_customer ? t8("You don't have the rights to edit this customer.")
+ : t8("You don't have the rights to edit this vendor.");
+
for my $bar ($::request->layout->get('actionbar')) {
$bar->add(
combobox => [
submit => [ '#form', { action => "CustomerVendor/save" } ],
checks => [ 'check_taxzone_and_ustid' ],
accesskey => 'enter',
+ disabled => $no_rights,
],
action => [
t8('Save and Close'),
submit => [ '#form', { action => "CustomerVendor/save_and_close" } ],
checks => [ 'check_taxzone_and_ustid' ],
+ disabled => $no_rights,
],
], # end of combobox "Save"
t8('Save and AP Transaction'),
submit => [ '#form', { action => "CustomerVendor/save_and_ap_transaction" } ],
checks => [ 'check_taxzone_and_ustid' ],
+ disabled => $no_rights,
]) x !!$self->is_vendor,
(action => [
t8('Save and AR Transaction'),
submit => [ '#form', { action => "CustomerVendor/save_and_ar_transaction" } ],
checks => [ 'check_taxzone_and_ustid' ],
+ disabled => $no_rights,
]) x !$self->is_vendor,
action => [
t8('Save and Invoice'),
submit => [ '#form', { action => "CustomerVendor/save_and_invoice" } ],
checks => [ 'check_taxzone_and_ustid' ],
+ disabled => $no_rights,
],
action => [
t8('Save and Order'),
submit => [ '#form', { action => "CustomerVendor/save_and_order" } ],
checks => [ 'check_taxzone_and_ustid' ],
+ disabled => $no_rights,
],
(action => [
t8('Save and RFQ'),
submit => [ '#form', { action => "CustomerVendor/save_and_rfq" } ],
checks => [ 'check_taxzone_and_ustid' ],
+ disabled => $no_rights,
]) x !!$self->is_vendor,
(action => [
t8('Save and Quotation'),
submit => [ '#form', { action => "CustomerVendor/save_and_quotation" } ],
checks => [ 'check_taxzone_and_ustid' ],
+ disabled => $no_rights,
]) x !$self->is_vendor,
], # end of combobox "Workflow"
confirm => t8('Do you really want to delete this object?'),
disabled => !$self->{cv}->id ? t8('This object has not been saved yet.')
: !$self->is_orphaned ? t8('This object has already been used.')
- : undef,
+ : $no_rights,
],
'separator',
},
customernumber => t8('Customer Number'),
},
- query => [
- ( salesman_id => SL::DB::Manager::Employee->current->id) x !$::auth->assert('customer_vendor_all_edit', 1),
- ],
);
}
);
}
+sub init_zugferd_settings {
+ return [
+ [ -1, t8('Use settings from client configuration') ],
+ @SL::ZUGFeRD::customer_settings,
+ ],
+}
+
sub _new_customer_vendor_object {
my ($self) = @_;
my $class = 'SL::DB::' . ($self->is_vendor ? 'Vendor' : 'Customer');
- return $class->new(
+ my $object = $class->new(
contacts => [],
shipto => [],
custom_variables => [],
);
+
+ $object->additional_billing_addresses([]) if $self->is_customer;
+
+ return $object;
}
sub _new_contact_object {
projects / kivitendo-erp.git / blobdiff