Opened 12 years ago
Closed 12 years ago
#8916 closed enhancement (fixed)
[Interface kit] Add BNotification constructor missing code.
| Reported by: | Prodito | Owned by: | leavengood |
|---|---|---|---|
| Priority: | normal | Milestone: | R1 |
| Component: | Kits/Interface Kit | Version: | R1/Development |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: | ||
| Platform: | All |
Description
When BNotification::Send is called the class is saved to a BMessage and sent to the Notification server, then the notification server re-create the BNotification object using the BNotification::BNotification(BMessage*) constructor, but the constructor code don't load _onClickRef and _onClickArgv from the BMessage which is used to run an app or a file when the user click on the notification.
Attachments (1)
Change History (6)
by , 12 years ago
| Attachment: | 0001-Patch-to-fix-BNotification-construct-from-BMessage.patch added |
|---|
comment:1 by , 12 years ago
| patch: | 0 → 1 |
|---|
comment:2 by , 12 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → in-progress |
comment:4 by , 12 years ago
Sorry I obviously dropped the ball on this one. I got involved in the Stripe CTF competition last week and that sucked up my whole weekend (I did capture the flag though!) I will do this for sure this weekend, along with other Haiku work.
comment:5 by , 12 years ago
| Resolution: | → fixed |
|---|---|
| Status: | in-progress → closed |
Patch applied in hrev44603. Thanks again and sorry for the delay!
Note:
See TracTickets
for help on using tickets.
Nice catch, and thanks for the patch. I'll apply it later today.