Skip to content

fixed Context.getSelectedDaysString Array casting to AttayList.#365

Open
tristanlifn wants to merge 1 commit into
FossifyOrg:mainfrom
tristanlifn:main
Open

fixed Context.getSelectedDaysString Array casting to AttayList.#365
tristanlifn wants to merge 1 commit into
FossifyOrg:mainfrom
tristanlifn:main

Conversation

@tristanlifn

@tristanlifn tristanlifn commented Jul 17, 2026

Copy link
Copy Markdown

Type of change(s)

  • Bug fix
  • Feature / enhancement
  • Infrastructure / tooling (CI, build, deps, tests)
  • Documentation

What changed and why

I am working on a feature for the clock app that utilizes the extensions.Context.getSelectedDaysString function, but i found that the type casting (on line 636) was crashing/hanging when debuging the feature. To test why, i made a hack fix, implementing said function in the file i was working in and tried to change it to a safe convertion. I have introduced this fix in this PR, it is a simple one line fix. I am not sure why this has never been a problem before, but this change should not break anything.

Tests performed

This change should have no impact on performance on any device.

Checklist

  • I read the contribution guidelines.
  • I manually tested my changes on device/emulator (not applicable).
  • I updated the "Unreleased" section in CHANGELOG.md (not applicable).
  • I have self-reviewed my pull request (no typos, formatting errors, etc.).
  • I understand every change in this pull request.

@tristanlifn
tristanlifn requested a review from naveensingh as a code owner July 17, 2026 18:00
tristanlifn added a commit to tristanlifn/Clock that referenced this pull request Jul 17, 2026
…e are more than zero groups. Removed hack fix for "getSelectedDaysString". Changed group toggle off when no alarms are on, not when all are on. changed commons version to 6.1.7 (see [PR FossifyOrg#365](FossifyOrg/commons#365))
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