Opened 14 years ago
Last modified 12 years ago
#145 new task
Reorganize ProxyFactory
Reported by: | adube | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | 1.8.0 |
Component: | server | Version: | trunk |
Keywords: | bis-1.8.0 | Cc: |
Description
current status
With the arrival of new the new "blank" service types such as "config" and "widget", the ProxyFactory was adjusted to be able to accept a
__blank__
value as service name in order to call the according new service types.
issue
The current way of determining which kind of 'blank' service type has to be called is done by looking at other parameters, such as 'osmwidgettype'. By default, setting nothing else calls the 'config' proxy. If other types of proxy were to be implemented, they would require to have unique parameters set in order to be properly called even though they would not necessarily require them.
solution
Instead of defining the blank value (see above) as service, each 'blank' type could have its unique value. For example :
- org_geoprisma_proxy_widget_WidgetProxy :
__widget__
- org_geoprisma_proxy_config_ConfigProxy :
__config__
- org_geoprisma_proxy_newone_NewOneProxy :
__newone__
- etc.
Then calling them in the ProxyFactory would could be done with a 'switch' statement on the 'osmservice' param value directly.
All widget using the current 'blank' method would require to be adjusted to this new method by using the new according value.
Change History (7)
comment:1 by , 13 years ago
Milestone: | → 1.2.0 |
---|
comment:2 by , 13 years ago
Milestone: | 1.2.0 → 1.4.0 |
---|
comment:4 by , 13 years ago
Keywords: | bis-1.2.0 added |
---|
comment:5 by , 13 years ago
Keywords: | bis-1.4.0 added; bis-1.2.0 removed |
---|---|
Milestone: | 1.2.0 → 1.4.0 |
Version: | → trunk |
No patch, pushed to 1.4.
comment:6 by , 13 years ago
Keywords: | bis-1.6.0 added; bis-1.4.0 removed |
---|---|
Milestone: | 1.4.0 → 1.6.0 |
No patch, pushed to 1.6.
comment:7 by , 12 years ago
Keywords: | bis-1.8.0 added; bis-1.6.0 removed |
---|---|
Milestone: | 1.6.0 → 1.8.0 |
No patch, pushed to 1.8.
No patch, pushed to 1.4.