Skip to content

fix(session): Add null check for boardId in createSession#7660

Merged
grnd-alt merged 1 commit intomainfrom
fixBoardIdNull
Feb 24, 2026
Merged

fix(session): Add null check for boardId in createSession#7660
grnd-alt merged 1 commit intomainfrom
fixBoardIdNull

Conversation

@solracsf
Copy link
Member

@solracsf solracsf commented Feb 19, 2026

Summary

Exception
OCA\Deck\Controller\SessionController::create(): Argument #1 ($boardId) must be of type int, null given

Remove createSession() from created() entirely and move the single call site to fetchData(), after loadBoardById has successfully resolved — guaranteeing this.id is a valid integer.

Checklist

  • Code is properly formatted
  • Sign-off message is added to all commits
  • Tests (unit, integration, api and/or acceptance) are included
  • Documentation (manuals or wiki) has been updated or is not required

@solracsf solracsf marked this pull request as ready for review February 19, 2026 17:36
@solracsf solracsf added bug frontend 3. to review javascript Pull requests that update Javascript code labels Feb 19, 2026
Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
@grnd-alt grnd-alt merged commit d1d4fbf into main Feb 24, 2026
36 checks passed
@grnd-alt grnd-alt deleted the fixBoardIdNull branch February 24, 2026 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review bug frontend javascript Pull requests that update Javascript code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants