hasToInvoice gets enabled with isActive
gitea/salix/pipeline/head This commit looks good
Details
gitea/salix/pipeline/head This commit looks good
Details
This commit is contained in:
parent
fcca5c5eab
commit
e8df57dae2
|
@ -8,6 +8,14 @@ export default class Controller extends Section {
|
||||||
|
|
||||||
const hasContactData = this.client.email || this.client.phone || this.client.mobile;
|
const hasContactData = this.client.email || this.client.phone || this.client.mobile;
|
||||||
const hasChangedTaxData = !orgData.isTaxDataChecked && this.client.isTaxDataChecked;
|
const hasChangedTaxData = !orgData.isTaxDataChecked && this.client.isTaxDataChecked;
|
||||||
|
|
||||||
|
const shouldInvoice = this.client.isActive && !this.client.hasToInvoice;
|
||||||
|
const clientActivation = this.client.isActive && (orgData.isActive != this.client.isActive);
|
||||||
|
if (shouldInvoice && clientActivation) {
|
||||||
|
this.client.hasToInvoice = true;
|
||||||
|
this.vnApp.showMessage(this.$t('Client invoices enabled'));
|
||||||
|
}
|
||||||
|
|
||||||
if (hasChangedTaxData && hasContactData)
|
if (hasChangedTaxData && hasContactData)
|
||||||
this.checkExistingClient();
|
this.checkExistingClient();
|
||||||
else this.save();
|
else this.save();
|
||||||
|
|
|
@ -50,6 +50,16 @@ describe('Client', () => {
|
||||||
expect(controller.save).not.toHaveBeenCalledWith();
|
expect(controller.save).not.toHaveBeenCalledWith();
|
||||||
expect(controller.checkExistingClient).toHaveBeenCalledWith();
|
expect(controller.checkExistingClient).toHaveBeenCalledWith();
|
||||||
});
|
});
|
||||||
|
|
||||||
|
it('should enable the hasToInvoice property any time the form activates the client with isActive', () => {
|
||||||
|
$scope.watcher.orgData.isActive = false;
|
||||||
|
controller.client.isActive = true;
|
||||||
|
controller.client.hasToInvoice = false;
|
||||||
|
|
||||||
|
controller.onSubmit();
|
||||||
|
|
||||||
|
expect(controller.client.hasToInvoice).toBe(true);
|
||||||
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
describe('checkExistingClient()', () => {
|
describe('checkExistingClient()', () => {
|
||||||
|
|
Loading…
Reference in New Issue