spikerj/spikersoft-issues
1
0
Fork 0
Code Issues 196 Pull Requests Actions Packages Projects Releases Wiki Activity

CourseModule constructor never assigns this.number — every module.number is undefined #809

New Issue
Open
opened 2026-07-23 04:33:41 +00:00 by spikerj · 0 comments
spikerj commented 2026-07-23 04:33:41 +00:00
Owner
Copy Link
Copy Source

Bug (found via coverage wave 3, spikersoft-angular PR #551).

CourseModule (projects/spikersoft/src/app/_components/classes/module.ts) declares a constructor parameter number: number and a public number: number field, but the constructor body never runs this.number = number; — it only assigns activity_title, activity_type, lessons, title.

Result: every CourseModule instance has number === undefined, even though classes.ts passes 1..10 for both cp101 and dbs101. The parameter is dead and the public field is permanently undefined.

Currently latent (no reader of module.number found in the classes dir), so no visible breakage — but any UI/logic that later orders or labels modules by .number would silently get undefined.

Fix: add this.number = number; to the constructor.

Pinned as a characterization test in PR #551 (classes.spec.ts) asserting the current undefined behavior and referencing this ticket; flip those assertions to expect 1..10 when fixed.

**Bug (found via coverage wave 3, spikersoft-angular PR #551).** `CourseModule` (`projects/spikersoft/src/app/_components/classes/module.ts`) declares a constructor parameter `number: number` and a `public number: number` field, but the constructor body never runs `this.number = number;` — it only assigns `activity_title`, `activity_type`, `lessons`, `title`. Result: **every** `CourseModule` instance has `number === undefined`, even though `classes.ts` passes `1..10` for both `cp101` and `dbs101`. The parameter is dead and the public field is permanently undefined. Currently latent (no reader of `module.number` found in the classes dir), so no visible breakage — but any UI/logic that later orders or labels modules by `.number` would silently get `undefined`. **Fix:** add `this.number = number;` to the constructor. Pinned as a characterization test in PR #551 (`classes.spec.ts`) asserting the current `undefined` behavior and referencing this ticket; flip those assertions to expect `1..10` when fixed.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
agentic
Actionable surfaced by the agentic architecture & security review
 bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 sonarqube
Code quality issue surfaced by SonarQube (bug / code smell)
 wontfix
This won't be fixed
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
spikerj (Joseph Spiker)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: spikerj/spikersoft-issues#809