Skip to content
This repository was archived by the owner on Mar 4, 2025. It is now read-only.

Commit 7112ada

Browse files
author
vikasrohit
committed
AS#102438220269693, Clean up lint errors for app/scripts/filepicker.js & angular-filepicker.js
-- Fixed lint errors
1 parent 8ecbf2a commit 7112ada

File tree

6 files changed

+24
-22
lines changed

6 files changed

+24
-22
lines changed

.eslintignore

+3
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
.gitignore
2+
assets/scripts/*.*
3+
report/**/*.*

app/directives/tc-fp-file-input/tc-fp-file-input.directive.js

+12-12
Original file line numberDiff line numberDiff line change
@@ -33,36 +33,36 @@ import _ from 'lodash'
3333
}
3434
// set extensions
3535
if (scope.fieldId.indexOf('ZIP') > -1) {
36-
scope.extensions = ".zip"
36+
scope.extensions = '.zip'
3737
} else if (scope.fieldId.indexOf('DESIGN_COVER') > -1) {
38-
scope.extensions = ".png,.jpeg,.jpg,.bmp"
38+
scope.extensions = '.png,.jpeg,.jpg,.bmp'
3939
}
4040

4141
// set default services
42-
scope.fpServices = scope.fpServices || "COMPUTER,GOOGLE_DRIVE,BOX,DROPBOX"
42+
scope.fpServices = scope.fpServices || 'COMPUTER,GOOGLE_DRIVE,BOX,DROPBOX'
4343
scope.fpContainer = CONSTANTS.FILE_PICKER_SUBMISSION_CONTAINER_NAME || 'submission-staging-dev'
4444

4545
// set max size
4646
scope.maxSize = 500*1024*1024
4747

48-
var key, value;
48+
var key, value
4949
/*
5050
*pass original event
5151
*/
5252
element.bind('change', function(event) {
53-
event.preventDefault()
54-
scope.onSuccess(event.originalEvent || event);
55-
$rootScope.$apply()
56-
});
57-
element = element.length ? element[0] : element;
53+
event.preventDefault()
54+
scope.onSuccess(event.originalEvent || event)
55+
$rootScope.$apply()
56+
})
57+
element = element.length ? element[0] : element
5858
for (key in attrs.$attr){
59-
value = attrs.$attr[key]
60-
element.setAttribute(value, attrs[key])
59+
value = attrs.$attr[key]
60+
element.setAttribute(value, attrs[key])
6161
}
6262
filepickerService.constructWidget(element)
6363

6464
scope.onSuccess = function (event) {
65-
debugger
65+
// debugger
6666
var fpFile = event.fpfile
6767
var _file = {
6868
name: scope.filename || fpFile.filename,

app/services/submissions.service.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -27,9 +27,9 @@ import angular from 'angular'
2727

2828
// uploadSubmissionFileToS3(response, response.data.files, files, progressCallback)
2929

30-
console.log(response);
30+
logger.debug(response)
3131

32-
processSubmission(response, progressCallback);
32+
processSubmission(response, progressCallback)
3333
})
3434
.catch(function(err) {
3535
logger.error('Could not get presigned url', err)
@@ -136,7 +136,7 @@ import angular from 'angular'
136136

137137
progressCallback.call(progressCallback, 'FINISH', 100)
138138

139-
return response;
139+
return response
140140
})
141141
.catch(function(err) {
142142
logger.error('Could not start processing', err)

app/submissions/submit-design-files/submit-design-files.controller.js

+3-4
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ import _ from 'lodash'
1212
if (!challengeToSubmitTo.challenge) { return }
1313

1414
var vm = this
15-
var files = {}
1615
var fileUploadProgress = {}
1716
vm.urlRegEx = new RegExp(/^(http(s?):\/\/)?(www\.)?[a-zA-Z0-9\.\-\_]+(\.[a-zA-Z]{2,3})+(\/[a-zA-Z0-9\_\-\s\.\/\?\%\#\&\=]*)?$/)
1817
vm.rankRegEx = new RegExp(/^[1-9]\d*$/)
@@ -149,15 +148,15 @@ import _ from 'lodash'
149148
vm.submissionsBody.data.fonts = processedFonts
150149
SubmissionsService.startSubmission(vm.submissionsBody, updateProgress)
151150
.then(function(newSubmission) {
152-
logger.debug("New Submission: ", newSubmission)
151+
logger.debug('New Submission: ', newSubmission)
153152
//SubmissionsService.processSubmission(newSubmission, updateProgress)
154153
})
155154
.then(function(processedSubmission) {
156-
logger.debug("Processed Submission: ", processedSubmission)
155+
logger.debug('Processed Submission: ', processedSubmission)
157156

158157
})
159158
.catch(function(err) {
160-
logger.error("Submission processing failed ", err)
159+
logger.error('Submission processing failed ', err)
161160
})
162161
}
163162

package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
"scripts": {
55
"build": "webpack --bail --progress --build --tc",
66
"start": "webpack-dev-server --history-api-fallback --host 0.0.0.0 --dev --tc --inline --progress --port 3000",
7-
"lint": "eslint --ignore-path .gitignore .",
7+
"lint": "eslint .",
88
"test": "karma start --tc --test"
99
},
1010
"devDependencies": {

webpack.config.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,8 @@ const config = require('appirio-tech-webpack-config')({
1919
},
2020
plugins: [
2121
new BowerWebpackPlugin({
22-
moduleDirectories: ["bower_components"],
23-
manifestFiles: "bower.json",
22+
moduleDirectories: ['bower_components'],
23+
manifestFiles: 'bower.json',
2424
includes: /.*/,
2525
excludes: [],
2626
searchResolveModulesDirectories: true

0 commit comments

Comments
 (0)