[GeoNetwork-devel] Creating child bug

Hi,

While woring on a specific GeoNetwork instance, I figured out a bug on the MD editing, using the current workflow :

  • create a child from a MD
  • edit the parent one
  • try to upload a file related to the current edited MD

The XSL code from metadata-iso19139.xsl will then generate a JS call to startFileUpload, but the xpath would resolve (maybe in some cases, depending on the XSL parser used ?) to the bad MD id. Instead of using the identifier from the current edited MD, it will use the one from the (last ?) child.

I Attached a patch which aims to fix this behaviour. I found out some other xpath in the current patched stylesheet which should be potentially modified, but need a more experienced insight on it, I’m not sure that modifying them all could be a good idea, and prefer wait for your feedback about it.

Cheers,


Pierre Mauduit

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
Tel : + 33 (0)4 79 44 44 92
http://www.camptocamp.com
pierre.mauduit@anonymised.com

(attachments)

metadata-iso19139.xsl.patch (1.28 KB)

Fixed :http://trac.osgeo.org/geonetwork/ticket/512
Thanks for reporting.

Ciao Pierre.

Francois

Le 15 avril 2011 15:36, Pierre Mauduit <pierre.mauduit@anonymised.com> a écrit :

Hi,
While woring on a specific GeoNetwork instance, I figured out a bug on the
MD editing, using the current workflow :
- create a child from a MD
- edit the parent one
- try to upload a file related to the current edited MD
The XSL code from metadata-iso19139.xsl will then generate a JS call to
startFileUpload, but the xpath would resolve (maybe in some cases, depending
on the XSL parser used ?) to the bad MD id. Instead of using the identifier
from the current edited MD, it will use the one from the (last ?) child.
I Attached a patch which aims to fix this behaviour. I found out some other
xpath in the current patched stylesheet which should be potentially
modified, but need a more experienced insight on it, I'm not sure that
modifying them all could be a good idea, and prefer wait for your feedback
about it.
Cheers,

--
Pierre Mauduit

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
Tel : + 33 (0)4 79 44 44 92
http://www.camptocamp.com
pierre.mauduit@anonymised.com

------------------------------------------------------------------------------
Benefiting from Server Virtualization: Beyond Initial Workload
Consolidation -- Increasing the use of server virtualization is a top
priority.Virtualization can reduce costs, simplify management, and improve
application availability and disaster protection. Learn more about boosting
the value of server virtualization. http://p.sf.net/sfu/vmware-sfdev2dev
_______________________________________________
GeoNetwork-devel mailing list
GeoNetwork-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geonetwork-devel
GeoNetwork OpenSource is maintained at
http://sourceforge.net/projects/geonetwork