Skip to content

Fix vue warnings#4934

Open
xb058t wants to merge 3 commits into
Ericsson:masterfrom
xb058t:fix-vue-warnings
Open

Fix vue warnings#4934
xb058t wants to merge 3 commits into
Ericsson:masterfrom
xb058t:fix-vue-warnings

Conversation

@xb058t

@xb058t xb058t commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

Fixed:

TheHeader.vue - [Vue Router warn]: Discarded invalid param(s) "endpoint" when navigating - removed explicit endpoint param, Vue Router 4.1.4+ inherits it automatically

CheckerMessageFilter.vue - [Vue warn]: Property "setSelectedItems" was accessed during render but is not defined on instance - was referencing a non-existent local function instead of baseSelectOptionFilter.setSelectedItems

SingleLineWidget.vue - [Vue warn]: Invalid prop: type check failed for prop "value". Expected Array, got Null/Object/Number - prop type was too strict, broadened to [Array, Number, Object]

RunList.vue - [Vue warn]: Failed to resolve component: AnalysisInfoDialog - component was used in template but never imported

Statistics.vue - [Vue Router warn]: can no longer be used directly inside - updated to Vue 3 slot pattern

Statistics.vue - [Vue warn]: Runtime directive used on component with non-element root node - v-fill-height was on (fragment root), moved to a wrapping

@xb058t xb058t requested a review from gulyasgergely902 as a code owner June 29, 2026 18:10
helpMessage: { type: String, default: null },
bus: { type: Object, required: true },
value: { type: Array, required: true },
value: { type: [ Array, Number, Object ], default: null },

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What was the reason setting it to accept arrays, numbers and objects? In the code this component is used 2 times and on both occasions it is getting a number so theoretically only numbers should be expected. Or did I miss something?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added it because of the Expected Array, got Object/Number/Null warning, but fixed the actual cause instead, getNumberOfFailedFiles() wasn't calling .toNumber(). value is just Number now.

RunSortType
} from "@cc/report-server-types";
import AnalysisInfoDialog from "@/components/AnalysisInfoDialog";

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This dialog was moved to the RunNameColumn itself where it gets activated. Because of this modification, the dialog can be removed from RunList entirely.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed it from RunList.vue entirely. Also cleaned up the unused openAnalysisInfoDialog prop in RunNameColumn.vue and ExpandedRun.vue.

@xb058t xb058t requested a review from gulyasgergely902 June 30, 2026 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants