YuhanXu / omnidroid

Automatically exported from code.google.com/p/omnidroid
Apache License 2.0
0 stars 0 forks source link

Add "Time Filter" to all Events #109

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
"Time is" filter should be applicable to all events.

Original issue reported on code.google.com by case.and...@gmail.com on 3 Jul 2010 at 5:32

GoogleCodeExporter commented 8 years ago

Original comment by case.and...@gmail.com on 4 Jul 2010 at 6:53

GoogleCodeExporter commented 8 years ago

Original comment by case.and...@gmail.com on 6 Jul 2010 at 2:19

GoogleCodeExporter commented 8 years ago
Trying to work on this but can't figure out how to add filters

Original comment by eligeler...@gmail.com on 6 Jul 2010 at 8:33

GoogleCodeExporter commented 8 years ago

Original comment by eligeler...@gmail.com on 6 Jul 2010 at 10:17

GoogleCodeExporter commented 8 years ago
Elig (What's your realname btw?),

These are actually event attributes.  They're stored in the DB using these 
lines:

    eventAttributeDbAdapter.insert(SMSReceivedEvent.ATTRIB_MESSAGE_TIME, eventIdSmsRec, dataTypeIdDate);
...
    eventAttributeDbAdapter.insert(PhoneRingingEvent.ATTRIBUTE_TIMESTAMP, eventIdPhoneRings, dataTypeIdDate);
...
etc.,

And of course that means they need to be implemented in the 
[INSERTEVENTNAME]Event.java files as well.

That may or may not be all that needs to happen.  A bit of experimenting I'm 
sure will need to be done.  If you can write a Wiki article about how to do 
this afterwards, it would be helpful as well.

Original comment by case.and...@gmail.com on 8 Jul 2010 at 3:07

GoogleCodeExporter commented 8 years ago
Issue 123 is a more generalized version of this bug.

Original comment by case.and...@gmail.com on 13 Jul 2010 at 5:45

GoogleCodeExporter commented 8 years ago

Original comment by eligeler...@gmail.com on 13 Jul 2010 at 10:38

GoogleCodeExporter commented 8 years ago
Merging this into the more general ticket since this is a subset of it and 
you're working on them both.

Original comment by case.and...@gmail.com on 13 Jul 2010 at 10:38