Opened 7 years ago

Closed 4 years ago

#23425 closed Cleanup/optimization (invalid)

Hide oracle specific GIS behaviour behind a feature flag

Reported by: Josh Smeaton Owned by: nobody
Component: GIS Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

There are several places where oracle is singled out in GIS. These checks should probably be pushed down into feature flags.

Some examples:

# GeoAggregate.as_sql
    if connection.ops.oracle:
            self.extra['tolerance'] = self.tolerance
# GeoQuery.get_aggregation
    if isinstance(aggregate, gis_aggregates.GeoAggregate):
        if not getattr(aggregate, 'is_extent', False) or connection.ops.oracle:
            self.extra_select_fields[alias] = GeomField()

Change History (2)

comment:1 Changed 7 years ago by Simon Charette

Triage Stage: UnreviewedAccepted

comment:2 Changed 4 years ago by Sergey Fedoseev

Resolution: invalid
Status: newclosed

git grep ops.oracle gives no results, so this ticket doesn't seem to be actual anymore.

Note: See TracTickets for help on using tickets.
Back to Top