[OpenLayers-Dev] Fwd: [OpenLayers-Commits] r6979 - sandbox/topp/almanac/lib/OpenLayers/Format

[Eric Lemoine]

Tim,

Wouldn't it make sense to add a "link" or "url" property to the
Feature class (I like "link" better)? It would make sense to me in
regards to the code we have in the 'delete' and 'update' methods of
Protocol/HTTP.js (where we assume that feature.url exists).

Thoughts?
--
Eric

---------- Forwarded message ----------
From:  <commits at openlayers.org>
Date: Sun, Apr 20, 2008 at 9:29 AM
Subject: [OpenLayers-Commits] r6979 - sandbox/topp/almanac/lib/OpenLayers/Format
To: commits at openlayers.org


Author: tschaub
 Date: 2008-04-20 03:29:40 -0400 (Sun, 20 Apr 2008)
 New Revision: 6979

 Modified:
   sandbox/topp/almanac/lib/OpenLayers/Format/Atom.js
 Log:
 atom requires href on link

 Modified: sandbox/topp/almanac/lib/OpenLayers/Format/Atom.js
 ===================================================================
 --- sandbox/topp/almanac/lib/OpenLayers/Format/Atom.js  2008-04-20
05:02:10 UTC (rev 6978)
 +++ sandbox/topp/almanac/lib/OpenLayers/Format/Atom.js  2008-04-20
07:29:40 UTC (rev 6979)
 @@ -208,11 +208,7 @@
                 feature.attributes.content = this.getChildValue(node);
             },
             "link": function(node, feature) {
 -                var href = this.getChildValue(node);
 -                if(!href) {
 -                    href = node.getAttribute("href");
 -                }
 -                feature.attributes.link = href;
 +                feature.attributes.link = node.getAttribute("href");
             },
             "id": function(node, feature) {
                 feature.fid = this.getChildValue(node);

 _______________________________________________
 Commits mailing list
 Commits at openlayers.org
 http://openlayers.org/mailman/listinfo/commits