Changeset 4785 for cows


Ignore:
Timestamp:
12/01/09 11:25:12 (11 years ago)
Author:
spascoe
Message:

Various comments on code structure. Grep for !TODO and !FIXME or see the diff.

Also an incomplete module for testing (iface_test_lib.py).

Location:
cows/trunk/cows
Files:
1 added
6 edited

Legend:

Unmodified
Added
Removed
  • cows/trunk/cows/service/imps/csmlbackend/csmlcommon.py

    r4695 r4785  
    77import csml 
    88import tempfile 
    9 from pylons import config, request, session  #config must have tmpfilebuffer and csmlstore values 
     9#!FIXME: modules in the service package shouldn't depend on pylons. 
     10from pylons import config #config must have tmpfilebuffer and csmlstore values 
    1011import ConfigParser 
    1112log = logging.getLogger(__name__) 
     
    5253    """ 
    5354    Map keyword arguments to a collection of layers. 
    54     Supports the retrieval of sets of layers according to arbitary 
     55    Supports the retrieval of sets of layers according to arbitrary 
    5556    keyword/value pairs. 
    5657    Implements  ILayerMapper  
     
    6162        self.connector=CSMLConnector() 
    6263     
     64    #!TODO: Should be _getInfo() as proposed in wms_csmllayer.  Should move to wfs_csmllayer as 
     65    #    this is the only place where it isn't overridden. 
    6366    def getInfo(self, feature): 
    6467        ''' given a csml feature, return basic info about the layer/feature/coverage 
     
    7679        return title, abstract 
    7780 
    78  
     81#!TODO: What's the point of this class?  It's just a container for a function. 
    7982class CSMLConnector(object): 
    8083    """ Contains common methods for CSML backend used across all WXS services """     
  • cows/trunk/cows/service/imps/csmlbackend/wms_csmllayer.py

    r4561 r4785  
    3535        super(CSMLwmsLayerMapper, self).__init__() 
    3636     
     37    #!TODO: (spascoe) Could be _getInfo() to make it clear it's internal 
    3738    def getInfo(self, feature): 
    3839        ''' given a csml feature, return info about the layer/feature 
  • cows/trunk/cows/service/imps/wms_layers.py

    r4695 r4785  
    1010from matplotlib import cm 
    1111import genutil 
     12#!FIXME: modules in the service package shouldn't depend on pylons! 
    1213from pylons import config  #config must have tmpfilebuffer and csmlstore values 
    1314from layers import LayerParser 
  • cows/trunk/cows/service/wms_iface.py

    r4266 r4785  
    3131 
    3232         
    33  
     33#!TODO: missing definitions: self.dimensions 
    3434class IwmsLayer(ILayer): 
    3535    """ 
  • cows/trunk/cows/service/wxs_iface.py

    r4274 r4785  
    4646        """ 
    4747        raise NotImplementedError 
    48      
     48 
    4949class ILayer(object): 
    5050    """ 
  • cows/trunk/cows/test/README

    r4613 r4785  
     1Package Organisation 
     2==================== 
     3 
     4COWS is divided into the following package hierarchy: 
     5 
     6 - cows: top-level package 
     7 - model: Classes reflecting the OWS common data model 
     8 - pylons: Components designed to work within the pylons web framework 
     9 - service: Components implementing OGC services and backends.  Note code in this package shouldn't reference pylons. 
     10 - test: The COWS test harness (see below) 
     11 - xml: Parsing capabilities xml into cows.model classes 
    112 
    213Testing COWS 
Note: See TracChangeset for help on using the changeset viewer.