gvSIG bugs #265
API change on View class
| Status: | New | % Done: | 0% | |
|---|---|---|---|---|
| Priority: | Normal | |||
| Assignee: | - | |||
| Category: | - | |||
| Target version: | - | |||
| Severity: | Add-on name: | Unknown | ||
| gvSIG version: | 1.12.0 | Add-on version: | ||
| gvSIG build: | 1204 | Add-on build: | ||
| Operative System: | Add-on resolve version: | |||
| Keywords: | Add-on resolve build: | |||
| Has patch: | No |
Description
initialize() method of View class has been changed to initialize(MapContext). This introduced some problems. See related thread in the TSC mailing list
There are some proposals related to this change:
- Remove initialize() from BaseView. A not compiling error will appear and it's easy to detect.
- Maintain two methods on View, initialize() and initialize(MapContext). initialize() should works as previously
- ...
Remember that to create a new View, the prefered way is used the ProjectDocumentFactory
History
#1
Updated by Francisco José Peñarrubia about 12 years ago
- Assignee set to Francisco Puga
Te lo asigno por si vas a hacer algo con esto, o lo ponemos sin versión para la rc1
#2
Updated by Francisco Puga about 12 years ago
- Assignee deleted (
Francisco Puga) - Target version deleted (
1.12.0-rc1)
Tras hacer algunas pruebas no me parece fácil de solucionar, sin incurrir en la posibilidad de meter más bugs.
Este problema del cambio de API afectará sobre todo a extensiones de terceros que estén intentando crear la Vista a mano. El método recomendado para crear una vista es usar el ProyectDocumentFactory y debería actualizarse a esto.
La forma de crear y asignar el MapControl y el MapContext es bastante confuso y difícil de trazar. Habría que valorar si es viable mejorarlo, sin romper compatibilidades. Quito la "versión prevista" pero lo dejo abierto por ahora como recordatorio por si trazamos algún plan de futuro ¿1.13?
#3
Updated by Álvaro Anguix over 10 years ago
- Project changed from Application: gvSIG desktop to | gvSIG desktop 1