View Single Post
Posts: 1,313 | Thanked: 2,978 times | Joined on Jun 2011 @ Finland
#1125
Originally Posted by slarti View Post
Only downside to this is that if you set an alarm with PM and define a "non-standard" snooze, disable the alarm in clock thus "transferring ownership" to the clock and then try modify it there without picking a "standard" snooze, the clock app crashes on save.
I could not reproduce this.

I tried:
- alarm in 1 minute
- default sound
- snooze 8 minutes
- used "Run rule's actions"
- disabled from clock application
- changed the time to couple minutes later
- saved

No crash.


I think that this is going to be quite rare and I would really like the option to be there. It would make it possible to, for example, have an 8 min + 8 min timer rule. The PM alarm does get deleted when stop is pressed even after a snooze. Maybe warn about this?
I'm not sure I understand.

Was a "not" missing in that sentence? Because I just noticed that in the new version, even after pressing "Stop" the alarm is not deleted from clock application. It just becomes inactive. I think in the previous version it was really deleted.

This is not a huge thing for me personally, as activating the alarm again does not create a new entry but instead the old one is reused.

But this definitely requires some testing, as I worry a bit that in some scenarios the alarm might be duplicated (ie. many entries with same title in clock application) which would not be nice.

I see that the sound attribute is now UNDEFINED if not set in PM and the sound seems to always default to "Clock 1.mp3". It doesn't follow the clock's default anymore? Is that what was causing it not being disablable (is that a word?)?
Yes... missing "sound" attribute caused the alarm to be undisablable (not sure that's a word either). It's unfortunate that it doesn't use the default now, but I think this is the lesser of two evils. One can always explicitly define the alarm sound.
__________________
My N9/N950 projects:
 

The Following User Says Thank You to ajalkane For This Useful Post: