Build and Test SynchWeb

LIMS-1163: Improve error message when token is invalid #505

Sign in to view logs

Triggered via pull request December 5, 2023 16:49

ndg63276

opened #702

improvement/LIMS-1163/improve-error-message-when-token-is-invalid

Status Failure

Total duration 1m 53s

Artifacts –

ci.yml

on: pull_request

Checkout, build, test and lint PHP code

JavaScript build, test and lint

Annotations

12 errors and 9 warnings

InvalidGlobal: src/Controllers/AuthenticationController.php#L164

src/Controllers/AuthenticationController.php:164:13: InvalidGlobal: Cannot use global scope here (see https://psalm.dev/046)

Checkout, build, test and lint PHP code

Process completed with exit code 2.

JavaScript build, test and lint: client/src/js/models/protein.js#L19

Do not access attributes directly. Use set() and get() instead

JavaScript build, test and lint: client/src/js/models/visit.js#L54

Do not access attributes directly. Use set() and get() instead

JavaScript build, test and lint: client/src/js/models/visit.js#L55

Do not access attributes directly. Use set() and get() instead

JavaScript build, test and lint: client/src/js/models/visit.js#L56

Do not access attributes directly. Use set() and get() instead

JavaScript build, test and lint: client/src/js/models/visit.js#L56

Do not access attributes directly. Use set() and get() instead

JavaScript build, test and lint: client/src/js/models/visit.js#L56

Do not access attributes directly. Use set() and get() instead

JavaScript build, test and lint: client/src/js/models/visit.js#L57

Do not access attributes directly. Use set() and get() instead

JavaScript build, test and lint: client/src/js/modules/cell/views/beamlines.js#L45

Move function declaration to function body root

JavaScript build, test and lint: client/src/js/modules/dc/models/autointegration.js#L16

Do not access attributes directly. Use set() and get() instead

JavaScript build, test and lint: client/src/js/modules/dc/models/autointegration.js#L19

Do not access attributes directly. Use set() and get() instead

JavaScript build, test and lint: client/src/js/modules/dc/views/queuebuilder.js#L27

Do not silence events

JavaScript build, test and lint: client/src/js/modules/dc/views/queuebuilder.js#L28

Do not silence events

JavaScript build, test and lint: client/src/js/modules/dc/views/queuebuilder.js#L158

Do not silence events

JavaScript build, test and lint: client/src/js/modules/shipment/collections/platetypes.js#L139

Do not silence events

JavaScript build, test and lint: client/src/js/views/search.js#L169

Use this.$ instead of $ in views

JavaScript build, test and lint: client/src/js/views/search.js#L170

Use this.$ instead of $ in views

JavaScript build, test and lint: client/src/js/views/search.js#L172

Use this.$ instead of $ in views

JavaScript build, test and lint: client/src/js/views/search.js#L185

Use this.$ instead of $ in views

JavaScript build, test and lint: client/src/js/views/search.js#L186

Use this.$ instead of $ in views