Frontmatter
| id | 7874 |
| title | Remove duplicate contains() method in Rectangle.mjs |
| state | Closed |
| labels | bugairefactoring |
| assignees | tobiu |
| createdAt | Nov 23, 2025, 12:07 PM |
| updatedAt | Nov 23, 2025, 12:13 PM |
| githubUrl | https://github.com/neomjs/neo/issues/7874 |
| author | tobiu |
| commentsCount | 0 |
| parentIssue | null |
| subIssues | [] |
| subIssuesCompleted | 0 |
| subIssuesTotal | 0 |
| blockedBy | [] |
| blocking | [] |
| closedAt | Nov 23, 2025, 12:13 PM |
Description The
containsmethod insrc/util/Rectangle.mjsis defined twice.this.constructor.includes.This causes code duplication and confusion.
Proposed Change
containsmethod to explicitly useRectangle.includes(this, other)instead ofthis.constructor.includesto match the style used inleavesSideand ensure correctness.Acceptance Criteria
containsis defined only once.containsdelegates to the staticincludesmethod.