gvSIG bugs #2833

"Delete" in GeoDB connection parameters

Added by Antonio Falciano over 9 years ago. Updated about 9 years ago.

Status:Closed% Done:

0%

Priority:NormalSpent time:-
Assignee:José Badía
Category:Database
Target version:2.2.0-2302 (rev. org.gvsig.desktop-2.0.89)
Severity:Minor Add-on version:
gvSIG version:2.1.0 Add-on build:
gvSIG build:2247 Add-on resolve version:
Operative System: Add-on resolve build:
Keywords:i18n, internationalization, translation Proyecto:
Has patch: Hito:
Add-on name:Unknown

geodb_connection_parameters_i18n.png (63.7 KB) Antonio Falciano, 09/22/2014 04:56 AM

1040

Associated revisions

Revision 42046
Added by José Badía about 9 years ago

fixes #2833
"Delete" in GeoDB connection parameters

History

#1 Updated by Álvaro Anguix about 9 years ago

  • Target version set to 2.2.0-2311-rc2
  • Assignee set to José Badía

#2 Updated by Joaquín del Cerro Murciano about 9 years ago

  • Status changed from New to Fixed

En el build 2269 (la final) a mi me salen ya traducidos.

#3 Updated by Álvaro Anguix about 9 years ago

Cierto, debería haberlo comprobado.

#4 Updated by Álvaro Anguix about 9 years ago

  • Status changed from Fixed to Closed

#5 Updated by Antonio Falciano about 9 years ago

Joaquín del Cerro Murciano wrote:

En el build 2269 (la final) a mi me salen ya traducidos.

Hi Joaquín,
you're right! Only the Delete button is not translated in the main connection parameters dialog in build 2269. Furthermore, there are many untranslated strings in the advanced parameters (MySQLServerExplorerParameters).

#6 Updated by Antonio Falciano about 9 years ago

  • Status changed from Closed to New

Hi Alvaro,
I'd leave it open because the i18n strings are not all translated.

#7 Updated by Joaquín del Cerro Murciano about 9 years ago

Antonio Falciano wrote:

Joaquín del Cerro Murciano wrote:

En el build 2269 (la final) a mi me salen ya traducidos.

Hi Joaquín,
you're right! Only the Delete button is not translated in the main connection parameters dialog in build 2269. Furthermore, there are many untranslated strings in the advanced parameters (MySQLServerExplorerParameters).

Uppps,
el boton "Delete" no solo es que no esta traducido, es que no hace nada, no tiene codigo asociado.

https://redmine.gvsig.net/redmine/projects/gvsig-desktop/repository/revisions/41633/entry/trunk/org.gvsig.desktop/org.gvsig.desktop.plugin/org.gvsig.geodb.app/org.gvsig.geodb.app.mainplugin/src/main/java/org/gvsig/geodb/vectorialdb/wizard/VectorialDBConnectionParamsDialog.java#L56

#8 Updated by Antonio Falciano about 9 years ago

Joaquín del Cerro Murciano wrote:

el boton "Delete" no solo es que no esta traducido, es que no hace nada, no tiene codigo asociado.

True, so it's better to remove it because there's the Cancel button yet.

#9 Updated by Álvaro Anguix about 9 years ago

  • Subject changed from Missing translations in GeoDB connection parameters to Remove "Delete" in GeoDB connection parameters

Cambio el asunto del bug

#10 Updated by Álvaro Anguix about 9 years ago

si se puede, en lugar de eliminar el botón delete, se le añadiría la funcionalidad que espera el usuario.

#11 Updated by Álvaro Anguix about 9 years ago

  • Subject changed from Remove "Delete" in GeoDB connection parameters to "Delete" in GeoDB connection parameters

#12 Updated by Joaquín del Cerro Murciano about 9 years ago

De cara al desarrollador que le meta mano.

Antes todo el interface de usuario relacionado con conexion a BBDD estaba en el plugin de GEODB; pero desde algunos otros sitios se precisaba acceder a estos paneles, ademas de que habia que corregir muchas cosas en ellos.

La gestion del pool de configuraciones de conexiones a BBDD del usuario la lleve a otras partes:

  • Por un lado la logica esta en DAL, en DataServerExplorerPool y su inplementacion.
  • Y por otro lado en interface de usuario para seleccionar y añadir cosas al pool esta en mapcontrol, en JDBCConnectionPanel.

Desde GeoDB, en VectorialDBConnectionParamsDialog, simplemente se incrusta el panel de JDBCConnectionPanel y se le añaden los botones debajo; y ahi es donde esta el boton Delete que no hace nada.
Habria que añadir en VectorialDBConnectionParamsDialog.initComponents el codigo y la traduccion al boton de forma similar a como estan los otros.

Al panel JDBCConnectionPanel, se accede a traves de la propiedad jdbcServerExplorer de VectorialDBConnectionParamsDialog, y este tiene un metodo "delete" que en principio parece que dberia bastar, pero creo que habra que retocar ese metodo delete para que refresque los campos tras borrar la conexion, posiblemente baste con invocar a un clear al final del metodo.

Comento todo esto por que hay un follon con estas clases, ya que cuando hice el vambio a lo nuevo no elimine las clases antiguas por si alguien las usaba, y ahora hay codigo duplicado, parte del viejo y el nuevo, y puede resultar lioso para saber donde hay que tocar.

#13 Updated by José Badía about 9 years ago

  • Status changed from New to Fixed

#14 Updated by Joaquín del Cerro Murciano about 9 years ago

  • Target version changed from 2.2.0-2311-rc2 to 2.2.0-2302 (rev. org.gvsig.desktop-2.0.89)

#15 Updated by Álvaro Anguix about 9 years ago

  • Status changed from Fixed to Closed

Parece que ya está corregido, aunque ahora faltaran las cadenas en italiano.
Además el botón Delete y el de Cancelar, ahora tienen la misma cadena en italiano.
Pero supongo que esto ya es tarea de traducciones, así que doy por cerrado este bug.

Also available in: Atom PDF