gvSIG bugs #312

Add table, change properties table and restore defaults, no effect / botón restore defaults sin efecto en ventana de propiedades de tabla a añadir

Added by Mario Fevre over 12 years ago. Updated over 11 years ago.

Status:Closed% Done:

0%

Priority:LowSpent time:-
Assignee:Juan Lucas Domínguez
Category:-
Target version:2.0.0-devel-2050
Severity: Add-on version:
gvSIG version:2.0.0 Add-on build:
gvSIG build:2043 Add-on resolve version:
Operative System:Windows Add-on resolve build:
Keywords: Proyecto:
Has patch:No Hito:
Add-on name:Document: Table (org.gvsig.app.document.table.app.mainplugin)

Description

En:
setting properties at "explorer_parameters" window (tested add a table), "restore defaults" button has no effect.

Es:
Al presionar el botón "retore defaults" en la ventana de propiedades del archivo a añadir (probado con tablas), no hay ningún efecto identificable.

detected testing
org.gvsig.app.document.table.app.mainplugin
Add table
Add table, change properties table and restore defaults

History

#1 Updated by Manuel Madrid about 12 years ago

  • Target version set to 2.0.0-rc1

#2 Updated by Joaquín del Cerro Murciano almost 12 years ago

  • Target version changed from 2.0.0-rc1 to 2.0.0-alpha6

#3 Updated by Juan Lucas Domínguez almost 12 years ago

  • Assignee set to Juan Lucas Domínguez

#4 Updated by Juan Lucas Domínguez almost 12 years ago

  • Target version changed from 2.0.0-alpha6 to 2.0.0-rc1

#5 Updated by Joaquín del Cerro Murciano almost 12 years ago

  • Priority changed from Normal to Low

#6 Updated by Juan Lucas Domínguez almost 12 years ago

  • Status changed from New to In progress

Joaquin and I discussed this and thought that it does not make sense to "restore defaults" because there are not default values available (the default values of the feature type are not useful here)

A simple solution is not showing the "Restore defaults" button. For example, we can add a parameter to the constructor:

public DynObjectEditor(DynObject parameters, boolean show_restore_defaults) {
...

And decide in each case if it makes sense to show that button.

#7 Updated by Cesar Ordiñana almost 12 years ago

Si, sino tiene sentido se quita el botón, al menos para este caso, y ya está.

En cuanto al constructor, para afectar lo menos posible yo añadiría un nuevo contructor que sea llamado por el antiguo con valor false en el segundo parámetro.

#8 Updated by Juan Lucas Domínguez almost 12 years ago

  • Status changed from In progress to Fixed
  • Target version changed from 2.0.0-rc1 to 2.0.0-devel-2050

gvsig-desktop:r38547
Removed "Restore defaults" button as it does not make much sense. Can be added explicitly in other cases.

#9 Updated by María Maluenda over 11 years ago

  • Status changed from Fixed to Closed

Closed in the gvSIG build 2051

Also available in: Atom PDF