reviewdog [eslint] report
reported by reviewdog 🐶
Findings (11)
frontend/src/app/shared/components/datepicker/wp-modal-date-picker/wp-modal-date-picker.component.ts|45 col 10| 'DateModalRelationsService' is defined but never used. Allowed unused vars must match /^/u.
frontend/src/app/shared/components/datepicker/wp-modal-date-picker/wp-modal-date-picker.component.ts|45 col 10| 'DateModalRelationsService' is defined but never used. Allowed unused vars must match /^/u.
frontend/src/app/shared/components/datepicker/wp-modal-date-picker/wp-modal-date-picker.component.ts|102 col 3| Lifecycle methods should not be empty
frontend/src/app/shared/components/datepicker/wp-modal-date-picker/wp-modal-date-picker.component.ts|125 col 44| 'instance' is defined but never used. Allowed unused args must match /^/u.
frontend/src/app/shared/components/datepicker/wp-modal-date-picker/wp-modal-date-picker.component.ts|125 col 44| 'instance' is defined but never used. Allowed unused args must match /^/u.
frontend/src/app/shared/components/datepicker/wp-modal-date-picker/wp-modal-date-picker.component.ts|153 col 7| Unsafe argument of type any
assigned to a parameter of type HTMLElement | null
.
frontend/src/app/shared/components/fields/edit/modal-with-turbo-content/modal-with-turbo-content.directive.ts|70 col 5| Unsafe call of an any
typed value.
frontend/src/app/shared/components/fields/edit/modal-with-turbo-content/modal-with-turbo-content.directive.ts|73 col 8| Unsafe member access .addEventListener on an any
value.
frontend/src/app/shared/components/fields/edit/modal-with-turbo-content/modal-with-turbo-content.directive.ts|80 col 5| Unsafe call of an any
typed value.
frontend/src/app/shared/components/fields/edit/modal-with-turbo-content/modal-with-turbo-content.directive.ts|83 col 8| Unsafe member access .removeEventListener on an any
value.
frontend/src/app/shared/components/fields/edit/modal-with-turbo-content/modal-with-turbo-content.directive.ts|85 col 1| More than 1 blank line not allowed.
Filtered Findings (0)
Annotations
github-actions / eslint
[eslint] frontend/src/app/shared/components/datepicker/wp-modal-date-picker/wp-modal-date-picker.component.ts#L45 <no-unused-vars>(https://eslint.org/docs/latest/rules/no-unused-vars)
'DateModalRelationsService' is defined but never used. Allowed unused vars must match /^_/u.
Raw output
{"ruleId":"no-unused-vars","severity":2,"message":"'DateModalRelationsService' is defined but never used. Allowed unused vars must match /^_/u.","line":45,"column":10,"nodeType":"Identifier","messageId":"unusedVar","endLine":45,"endColumn":35}
github-actions / eslint
[eslint] frontend/src/app/shared/components/datepicker/wp-modal-date-picker/wp-modal-date-picker.component.ts#L45 <@typescript-eslint/no-unused-vars>(https://typescript-eslint.io/rules/no-unused-vars)
'DateModalRelationsService' is defined but never used. Allowed unused vars must match /^_/u.
Raw output
{"ruleId":"@typescript-eslint/no-unused-vars","severity":2,"message":"'DateModalRelationsService' is defined but never used. Allowed unused vars must match /^_/u.","line":45,"column":10,"nodeType":null,"messageId":"unusedVar","endLine":45,"endColumn":35}
github-actions / eslint
[eslint] frontend/src/app/shared/components/datepicker/wp-modal-date-picker/wp-modal-date-picker.component.ts#L102-L104 <@angular-eslint/no-empty-lifecycle-method>(https://github.com/angular-eslint/angular-eslint/blob/main/packages/eslint-plugin/docs/rules/no-empty-lifecycle-method.md)
Lifecycle methods should not be empty
Raw output
{"ruleId":"@angular-eslint/no-empty-lifecycle-method","severity":2,"message":"Lifecycle methods should not be empty","line":102,"column":3,"nodeType":"MethodDefinition","messageId":"noEmptyLifecycleMethod","endLine":104,"endColumn":4,"suggestions":[{"messageId":"suggestRemoveLifecycleMethod","fix":{"range":[1352,3847],"text":",\n ViewChild,\n} from '@angular/core';\nimport { I18nService } from 'core-app/core/i18n/i18n.service';\nimport { TimezoneService } from 'core-app/core/datetime/timezone.service';\nimport { DayElement } from 'flatpickr/dist/types/instance';\nimport flatpickr from 'flatpickr';\nimport { ApiV3Service } from 'core-app/core/apiv3/api-v3.service';\nimport { DateModalRelationsService } from 'core-app/shared/components/datepicker/services/date-modal-relations.service';\nimport { onDayCreate } from 'core-app/shared/components/datepicker/helpers/date-modal.helpers';\nimport { WeekdayService } from 'core-app/core/days/weekday.service';\nimport { FocusHelperService } from 'core-app/shared/directives/focus/focus-helper';\nimport { DeviceService } from 'core-app/core/browser/device.service';\nimport { DatePicker } from '../datepicker';\nimport { UntilDestroyedMixin } from 'core-app/shared/helpers/angular/until-destroyed.mixin';\nimport { PathHelperService } from 'core-app/core/path-helper/path-helper.service';\nimport { populateInputsFromDataset } from 'core-app/shared/components/dataset-inputs';\nimport { fromEvent } from 'rxjs';\nimport { filter } from 'rxjs/operators';\n\n@Component({\n selector: 'op-wp-modal-date-picker',\n template: `\n <input\n id=\"flatpickr-input\"\n #flatpickrTarget\n hidden>\n `,\n changeDetection: ChangeDetectionStrategy.OnPush,\n styleUrls: [\n '../styles/datepicker.modal.sass',\n ],\n})\nexport class OpWpModalDatePickerComponent extends UntilDestroyedMixin implements AfterViewInit {\n @Input() public ignoreNonWorkingDays:boolean;\n @Input() public scheduleManually:boolean;\n\n @Input() public startDate:Date;\n @Input() public dueDate:Date;\n\n @Input() public isSchedulable:boolean = true;\n @Input() public minimalSchedulingDate:Date|null;\n\n @Input() fieldName:string = '';\n\n @ViewChild('flatpickrTarget') flatpickrTarget:ElementRef;\n\n private datePickerInstance:DatePicker;\n\n constructor(\n readonly injector:Injector,\n readonly cdRef:ChangeDetectorRef,\n readonly apiV3Service:ApiV3Service,\n readonly I18n:I18nService,\n readonly timezoneService:TimezoneService,\n readonly deviceService:DeviceService,\n readonly weekdayService:WeekdayService,\n readonly focusHelper:FocusHelperService,\n readonly pathHelper:PathHelperService,\n readonly elementRef:ElementRef,\n ) {\n super();\n populateInputsFromDataset(this);\n }\n\n "},"desc":"Remove lifecycle method"}]}
github-actions / eslint
[eslint] frontend/src/app/shared/components/datepicker/wp-modal-date-picker/wp-modal-date-picker.component.ts#L125 <no-unused-vars>(https://eslint.org/docs/latest/rules/no-unused-vars)
'instance' is defined but never used. Allowed unused args must match /^_/u.
Raw output
{"ruleId":"no-unused-vars","severity":2,"message":"'instance' is defined but never used. Allowed unused args must match /^_/u.","line":125,"column":44,"nodeType":"Identifier","messageId":"unusedVar","endLine":125,"endColumn":52}
github-actions / eslint
[eslint] frontend/src/app/shared/components/datepicker/wp-modal-date-picker/wp-modal-date-picker.component.ts#L125 <@typescript-eslint/no-unused-vars>(https://typescript-eslint.io/rules/no-unused-vars)
'instance' is defined but never used. Allowed unused args must match /^_/u.
Raw output
{"ruleId":"@typescript-eslint/no-unused-vars","severity":2,"message":"'instance' is defined but never used. Allowed unused args must match /^_/u.","line":125,"column":44,"nodeType":null,"messageId":"unusedVar","endLine":125,"endColumn":52}
github-actions / eslint
[eslint] frontend/src/app/shared/components/datepicker/wp-modal-date-picker/wp-modal-date-picker.component.ts#L153 <@typescript-eslint/no-unsafe-argument>(https://typescript-eslint.io/rules/no-unsafe-argument)
Unsafe argument of type `any` assigned to a parameter of type `HTMLElement | null`.
Raw output
{"ruleId":"@typescript-eslint/no-unsafe-argument","severity":2,"message":"Unsafe argument of type `any` assigned to a parameter of type `HTMLElement | null`.","line":153,"column":7,"nodeType":"MemberExpression","messageId":"unsafeArgument","endLine":153,"endColumn":41}
github-actions / eslint
[eslint] frontend/src/app/shared/components/fields/edit/modal-with-turbo-content/modal-with-turbo-content.directive.ts#L70-L73 <@typescript-eslint/no-unsafe-call>(https://typescript-eslint.io/rules/no-unsafe-call)
Unsafe call of an `any` typed value.
Raw output
{"ruleId":"@typescript-eslint/no-unsafe-call","severity":2,"message":"Unsafe call of an `any` typed value.","line":70,"column":5,"nodeType":"MemberExpression","messageId":"unsafeCall","endLine":73,"endColumn":24}
github-actions / eslint
[eslint] frontend/src/app/shared/components/fields/edit/modal-with-turbo-content/modal-with-turbo-content.directive.ts#L73 <@typescript-eslint/no-unsafe-member-access>(https://typescript-eslint.io/rules/no-unsafe-member-access)
Unsafe member access .addEventListener on an `any` value.
Raw output
{"ruleId":"@typescript-eslint/no-unsafe-member-access","severity":2,"message":"Unsafe member access .addEventListener on an `any` value.","line":73,"column":8,"nodeType":"Identifier","messageId":"unsafeMemberExpression","endLine":73,"endColumn":24}
github-actions / eslint
[eslint] frontend/src/app/shared/components/fields/edit/modal-with-turbo-content/modal-with-turbo-content.directive.ts#L80-L83 <@typescript-eslint/no-unsafe-call>(https://typescript-eslint.io/rules/no-unsafe-call)
Unsafe call of an `any` typed value.
Raw output
{"ruleId":"@typescript-eslint/no-unsafe-call","severity":2,"message":"Unsafe call of an `any` typed value.","line":80,"column":5,"nodeType":"MemberExpression","messageId":"unsafeCall","endLine":83,"endColumn":27}
github-actions / eslint
[eslint] frontend/src/app/shared/components/fields/edit/modal-with-turbo-content/modal-with-turbo-content.directive.ts#L83 <@typescript-eslint/no-unsafe-member-access>(https://typescript-eslint.io/rules/no-unsafe-member-access)
Unsafe member access .removeEventListener on an `any` value.
Raw output
{"ruleId":"@typescript-eslint/no-unsafe-member-access","severity":2,"message":"Unsafe member access .removeEventListener on an `any` value.","line":83,"column":8,"nodeType":"Identifier","messageId":"unsafeMemberExpression","endLine":83,"endColumn":27}
github-actions / eslint
[eslint] frontend/src/app/shared/components/fields/edit/modal-with-turbo-content/modal-with-turbo-content.directive.ts#L85-L86 <no-multiple-empty-lines>(https://eslint.org/docs/latest/rules/no-multiple-empty-lines)
More than 1 blank line not allowed.
Raw output
{"ruleId":"no-multiple-empty-lines","severity":2,"message":"More than 1 blank line not allowed.","line":85,"column":1,"nodeType":"Program","messageId":"consecutiveBlank","endLine":86,"endColumn":1,"fix":{"range":[2983,2984],"text":""}}