Andal.LND/edx-platform
4
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

SOL-1225

Browse Source
This commit is contained in:
asadiqbal
2015-10-06 14:52:38 +05:00
parent 87e3988c6a
commit 0b28fd23f8
4 changed files with 8 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
cms/static/js/certificates/models/signatory.js
View File

@@ -33,11 +33,6 @@ function(_, str, Backbone, BackboneRelational, gettext) {
validate: function(attrs) {
var errors = null;
if(_.has(attrs, 'name') && attrs.name.length > 40) {
errors = _.extend({
'name': gettext('Signatory name should not be more than 40 characters long.')
}, errors);
}
if(_.has(attrs, 'title')){
var title = attrs.title;
var lines = title.split(/\r\n|\r|\n/);
@@ -46,13 +41,6 @@ function(_, str, Backbone, BackboneRelational, gettext) {
'title': gettext('Signatory title should span over maximum of 2 lines.')
}, errors);
}
else if ((lines.length > 1 && (lines[0].length > 53 && lines[1].length > 53)) ||
(lines.length === 1 && title.length > 106)) {
errors = _.extend({
'title': gettext('Signatory title should have maximum of 40 characters per line.')
}, errors);
}
}
if (errors !== null){
return errors;

2
cms/static/js/certificates/spec/views/certificate_details_spec.js
View File

@@ -252,7 +252,7 @@ function(_, Course, CertificatesCollection, CertificateModel, CertificateDetails
});
setValuesToInputs(this.view, {
inputSignatoryTitle: 'This is a certificate signatory title that has waaaaaaay more than 106 characters, in order to cause an exception.'
inputSignatoryTitle: 'Signatory Title \non three \nlines'
});
setValuesToInputs(this.view, {

6
cms/static/js/certificates/spec/views/certificate_editor_spec.js
View File

@@ -228,10 +228,10 @@ function(_, Course, CertificateModel, SignatoryModel, CertificatesCollection, Ce
}
);
it('signatories should not save when fields have too many characters per line', function() {
it('signatories should save when fields have too many characters per line', function() {
this.view.$(SELECTORS.addSignatoryButton).click();
setValuesToInputs(this.view, {
inputCertificateName: 'New Certificate Name'
inputCertificateName: 'New Certificate Name that has too many characters without any limit'
});
setValuesToInputs(this.view, {
@@ -243,7 +243,7 @@ function(_, Course, CertificateModel, SignatoryModel, CertificatesCollection, Ce
});
this.view.$(SELECTORS.saveCertificateButton).click();
expect(this.view.$('.certificate-edit-error')).toHaveClass('is-shown');
expect(this.view.$('.certificate-edit-error')).not.toHaveClass('is-shown');
});
it('signatories should not save when title span on more than 2 lines', function() {