GeoTools

Temporal filter support

Details

  • Type: New Feature New Feature
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 8.0-M1
  • Component/s: main
  • Labels:
    None

Description

Add temporal filter support based on the filter 2.0 specification. This involves:

  • Adding filter model classes for all the temporal operations
  • Updating FilterFactory and friends
  • Updating FilterVisitor and all implementations

Change proposal:

  1. Text File
    GEOT-2367.patch
    05/Jun/11 10:54 PM
    189 kB
    Justin Deoliveira

Issue Links

is depended upon by

New Feature - A new feature of the product, which has yet to be developed. GEOS-2688 Filter 2.0 support

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

New Feature - A new feature of the product, which has yet to be developed. GEOT-2368 Filter 2.0 xml bindings

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Justin Deoliveira made changes -
Field Original Value New Value
Link This issue is depended upon by GEOT-2368 [ GEOT-2368 ]
Justin Deoliveira made changes -
Link This issue is depended upon by GEOS-2688 [ GEOS-2688 ]
Hide
Permalink
Jody Garnett added a comment -
And details here Justin? Right now the bug report is not complete as I don't know what the temporal filter classes are.
Show
Jody Garnett added a comment - And details here Justin? Right now the bug report is not complete as I don't know what the temporal filter classes are.
Jody Garnett made changes -
Fix Version/s 2.7-M0 [ 15891 ]
Jody Garnett made changes -
Fix Version/s 2.7-M1 [ 16521 ]
Fix Version/s 2.7-M0 [ 15891 ]
Jody Garnett made changes -
Fix Version/s 2.7-RC1 [ 16571 ]
Fix Version/s 2.7-M1 [ 16521 ]
Jody Garnett made changes -
Fix Version/s 2.7-M3 [ 16572 ]
Fix Version/s 2.7-M2 [ 16571 ]
Justin Deoliveira made changes -
Fix Version/s 2.7-M4 [ 16726 ]
Fix Version/s 2.7-M3 [ 16572 ]
Andrea Aime made changes -
Fix Version/s 2.7-beta1 [ 16974 ]
Fix Version/s 2.7-M4 [ 16726 ]
Justin Deoliveira made changes -
Fix Version/s 2.7-RC1 [ 17076 ]
Fix Version/s 2.7-beta1 [ 16974 ]
Justin Deoliveira made changes -
Fix Version/s 2.7.0 [ 17109 ]
Fix Version/s 2.7-RC1 [ 17076 ]
Justin Deoliveira made changes -
Fix Version/s 2.7.1 [ 17255 ]
Fix Version/s 2.7.0 [ 17109 ]
Justin Deoliveira made changes -
Fix Version/s 2.7.2 [ 17359 ]
Fix Version/s 2.7.1 [ 17255 ]
Jody Garnett made changes -
Component/s main [ 10810 ]
Component/s core filter [ 11601 ]
Justin Deoliveira made changes -
Summary Add temporal filter classes. Temporal filter support
Fix Version/s 8.0-M1 [ 17333 ]
Fix Version/s 2.7.2 [ 17359 ]
Description Add temporal filter classes from the filter 2.0 specification. Add temporal filter support based on the filter 2.0 specification. This involves:

* Adding filter model classes for all the temporal operations
* Updating FilterFactory and friends
* Updating FilterVisitor and all implementations
Hide
Permalink
Justin Deoliveira added a comment -
Here is the initial patch for this one.
Show
Justin Deoliveira added a comment - Here is the initial patch for this one.
Justin Deoliveira made changes -
Attachment GEOT-2367.patch [ 55427 ]
Jody Garnett made changes -
Description Add temporal filter support based on the filter 2.0 specification. This involves:

* Adding filter model classes for all the temporal operations
* Updating FilterFactory and friends
* Updating FilterVisitor and all implementations 		Add temporal filter support based on the filter 2.0 specification. This involves:

* Adding filter model classes for all the temporal operations
* Updating FilterFactory and friends
* Updating FilterVisitor and all implementations

Change proposal:

* http://docs.codehaus.org/display/GEOTOOLS/Temporal+Filters
Justin Deoliveira made changes -
Status Open [ 1 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Hide
Permalink
Jody Garnett added a comment -
Not actually fixed until we have updated to docs / completed the tasks etc..
Show
Jody Garnett added a comment - Not actually fixed until we have updated to docs / completed the tasks etc..
Jody Garnett made changes -
Resolution Fixed [ 1 ]
Status Resolved [ 5 ] Reopened [ 4 ]
Assignee Justin Deoliveira [ jdeolive ] Jody Garnett [ jgarnett ]
Hide
Permalink
Jody Garnett added a comment -
Docs available for your review; direct use of DefaultInstance and DefaultPeriod were a bit rough. Perhaps we can make a "Temporal" utility class for main to make creating these things a bit easier.
Show
Jody Garnett added a comment - Docs available for your review; direct use of DefaultInstance and DefaultPeriod were a bit rough. Perhaps we can make a "Temporal" utility class for main to make creating these things a bit easier.
Jody Garnett made changes -
Assignee Jody Garnett [ jgarnett ] Justin Deoliveira [ jdeolive ]
Hide
Permalink
Justin Deoliveira added a comment -
Docs look good... and please no not another utility class :) I would rather just make DefaultInstant and DefaultPeriod easier to use by adding constructors that take dates... rather than Position instances.
Show
Justin Deoliveira added a comment - Docs look good... and please no not another utility class :) I would rather just make DefaultInstant and DefaultPeriod easier to use by adding constructors that take dates... rather than Position instances.
Hide
Permalink
Jody Garnett added a comment -
That is fine as well; it is just they are super scary to use right now. So does it just depend who does the work first :-)
Show
Jody Garnett added a comment - That is fine as well; it is just they are super scary to use right now. So does it just depend who does the work first :-)
Hide
Permalink
Justin Deoliveira added a comment -
Super scary? /me rolls eyes. It is a direct api Jody. Please don't go adding things that are not necessary. You really have a bad tendency to do this imo. In my opinion that makes the api way harder to use and understand than going through a direct route, even if it does involve one extra class in between.
Show
Justin Deoliveira added a comment - Super scary? /me rolls eyes. It is a direct api Jody. Please don't go adding things that are not necessary. You really have a bad tendency to do this imo. In my opinion that makes the api way harder to use and understand than going through a direct route, even if it does involve one extra class in between.
Hide
Permalink
Jody Garnett added a comment -
Let me just say it can be made easier - by adding additional constructors.
Show
Jody Garnett added a comment - Let me just say it can be made easier - by adding additional constructors.
Hide
Permalink
Jody Garnett added a comment -
Closing off issues prior to release; going to see if the sub tasks stay open.
Show
Jody Garnett added a comment - Closing off issues prior to release; going to see if the sub tasks stay open.
Jody Garnett made changes -
Status Reopened [ 4 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]

People

  • Assignee:
    Justin Deoliveira
    Reporter:
    Justin Deoliveira
Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: