Changeset 7256

Timestamp:

05/22/08 13:57:58 (6 months ago)

Author:

sbenthall

Message:

Save no longer sends unnecessary read() request; instead pulls fids for features right from the response from protocol.commit(). protocol.commit() now passes along more normalized information back to the callback passed to it. (Later this can be augmented to include full features).

Also, the WFS protocol now has its read request name parameterized out (in anticipation of WFSV protocol)

Files:

• sandbox/vector-behavior/lib/OpenLayers/Protocol/WFS/v1_0_0.js (modified) (3 diffs)
• sandbox/vector-behavior/lib/OpenLayers/Strategy/Save.js (modified) (1 diff)

sandbox/vector-behavior/lib/OpenLayers/Protocol/WFS/v1_0_0.js

@@ -96 +96 @@
-    
-    /**
+     * Property: readRequestName
+     * {String} The tag name for read requests.
+     */
+    readRequestName: "wfs:GetFeature",
+    
+    /**
-     * Constructor: OpenLayers.Protocol.WFS
-     * A class for giving layers WFS protocol.
@@ -153 +159 @@
-        OpenLayers.Util.applyDefaults(params, this.params);
-        
-"wfs:GetFeature"
+this.readRequestName
-        
-        var query = this.format.createElementNSPlus("wfs:Query", {
@@ -298 +304 @@
-        var data = this.format.write(root);
-        
-        var callback;
-        if(options){
-            callback = options.callback || function(){};
-        }
+        
+        var wfstCallback = function(request){
+            //Insert results
+            var inserts = [];         
+            var insertResult = request.responseXML.getElementsByTagName("InsertResult")[0];
+            var featureIdNodes = insertResult.getElementsByTagName("FeatureId");   
+            
+            for (var index = 0; index < featureIdNodes.length; index++) {
+                
+                featureId = featureIdNodes[index];
+                 
+                if (featureId) {
+                    var fid = this.format.getAttributeNS(featureId, "", "fid");
+                    inserts.push(fid);
+                }
+            }
+            
+            //Transaction result
+            var transactionResult = request.responseXML.getElementsByTagName("TransactionResult")[0];
+            var status = transactionResult.getElementsByTagName("Status")[0];
+            
+            if(status.getElementsByTagName("SUCCESS").length > 0){
+                callback({inserts : inserts, success : true});    
+            } else {
+                callback({success : false, request : request});
+            }
+           
+        }
+        
-        
-        return OpenLayers.Request.POST({
-            url: this.url,
-            data: data,
-
+
+
-        });      
-    },

sandbox/vector-behavior/lib/OpenLayers/Strategy/Save.js

@@ -88 +88 @@
-        var strategy = this;
-        
-reques
+objec
-            var state;
-            
-
-
-
-
-
-
+
+
+
+
+
+
+
+
+
+
+
+
-                }
-                
-
-
-
-
-
-
-
-
-
-
-
-
-
+
-                
-
-
-
-
-
+
+
+
+
+
+
+
+
+
+
-                }
-            }
-            
-            if(inserts[0] != "none"){
-                var fidFilter = new OpenLayers.Filter.FeatureId({fids: inserts});
-
-                params = {
-                    filter: fidFilter,       
-                    srsName: strategy.layer.projection.getCode()
-                };
-            
-                strategy.layer.protocol.read({
-                    params: params,
-                    callback: function(resp) {
-                        strategy.remove();
-                        strategy.merge(resp.features);
-                    },
-                    scope: this
-                });    
-            }
-        };