clone layer et feature

Version imprimable

03/04/2014, 10h49
cmail

clone layer et feature

Sur ma page de test, la couche Commune du viewer2 est obtenue par clonage de la couche Commune de viewer1.
Si on clique sur les feature de la couche Commune de viewer1, on affiche un popup.
Par contre, le clic sur les feature de la couche Commune de viewer2 ne déclenche aucune action. Pourquoi? Et comment rendre "sensibles" ces feature?
03/04/2014, 18h29
acarme

Bonjour,

le "pourquoi" de ça ne marche pas est difficile à expliquer, mais ce n'est pas vraiment la bonne façon de faire dans la mesure où la couche a été créée pour un viewer donné (viewer1) et clonée dans un autre viewer..

La bonne façon de faire serait :
1. créer le layer dans le viewer1
2. créer un layer vector "vide" dans le viewer2
3. peupler ce layer vector avec des feature créés à partir des feature du layer du viewer1
03/04/2014, 19h26
cmail

En suivant (je crois) tes conseils, j'obtiens cette nouvelle page de test.

Mais le constat est le même, il me semble.

Christian,

tu as bien saisi l'idée d'Anaïs, mais tu n'as pas poussé sa logique jusqu'au bout : il ne faut pas non plus cloner les features, mais les recréer à la main.

Je remplacerais donc :

Code:

var feature = this.features[i].clone();

par
Code:

1 2 3 4 5 6 var feature= new OpenLayers.Feature.Vector({ geometry: this.features[i].geometry.clone(), // on s'autorise quand même à cloner la Géométrie attributes : this.features[i].attributes.clone(), // idem pour les attributs style: this.features[i].style.clone() // idem pour le style }) ;
on ne clone donc plus que la partie relative aux informations utilisateurs portées par les objets (qui doit être copiée).

04/04/2014, 17h02
cmail

Sur ma page de test reprenant tes directives, j'ai un retour erreur : "this.features[i].attributes.clone is not a function".
Je ne vois pas de faute primaire de syntaxe????
08/04/2014, 13h09
gcebelieu
En fait, je me suis emballé sur l'existence d'un méthode clone pour n'importe quel objet javascript, ce qui n'est pas le cas.
Du coup, il faut cloner l'objet représenté par la propriété attributes "à la main".
en remplaçant :
Code:

1 2 3 attributes : this.features[i].attributes.clone() style: this.features[i].style.clone()
par
Code:

1 2 3 attributes : JSON.parse(JSON.stringify(this.features[i].attributes)), style: JSON.parse(JSON.stringify(this.features[i].style))
08/04/2014, 14h59
cmail

J'ai fait le changement sur ma page de test... Mais cela déclenche une erreur : c.geometry.getBounds is not a function sur l'API... Dans ces cas d'erreur, je suis assez perdu pour m'en sortir...:calim2:
08/04/2014, 15h26
gcebelieu
Christian,

visiblement le clone sur la geometry des features est aussi déficient. Du coup, essaye en faisant le clone à la main aussi sur celle-ci :
Code:

1 2 geometry: JSON.parse(JSON.stringify(this.features[i].geometry));
08/04/2014, 17h06
cmail

Avant d'envoyer ma réponse précédente, j'avais bien essayé ce "geometry: JSON.parse(JSON.stringify(this.features[i].geometry)), " mais j'avais l'erreur: "cyclic object value" sur cette même ligne.... Alors je m'étais dit que j'allais trop loin avec mon petit velo...
Mais puisque tu me le demandes, je viens de réécrire cette ligne à la place de "geometry: this.features[i].geometry.clone()," pour que tu voyes ce que cela donne...

Bon, visiblement, cloner avec JSON ne marche pas non plus, et la methode clone de OpenLayers.Geometry.XXX est bugguée.

On va essayer ceci pour faire disparaître le message "this.features[i].attributes.clone is not a function" :

1. tu remets :
Code:

1 2 geometry: this.features[i].geometry.clone(),
2. tu rajoutes ce code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 OpenLayers.Geometry.Collection.calculateBounds= function() { this.bounds = null; var bounds = new OpenLayers.Bounds(); var components = this.components; if (components) { for (var i=0, len=components.length; i<len; i++) { if (components[i].getBounds) bounds.extend(components[i].getBounds()); } } // to preserve old behavior, we only set bounds if non-null // in the future, we could add bounds.isEmpty() if (bounds.left != null && bounds.bottom != null && bounds.right != null && bounds.top != null) { this.setBounds(bounds); } },
au début de ta balise script.

3. tu me tiens au courant ;)

11/04/2014, 12h35
cmail

Résultat toujours sur ma page de test.
Erreur : c.geometry.getBounds is not a function dans /GeoportalExtended.js

Mais je ne sais si j'ai bien compris ce que tu me demandais de faire car la virgule en fin de ton code 2 me laisse perplexe....
30/04/2014, 15h07
cmail

Et mes clones?

Bonjour Christian,

un long moment de silence qui débouche sur des bonnes nouvelles :

1. le problème de l'erreur "getBounds is not a function" :

cela vient du fait que l'on n'appelle pas correctement le constructeur de OpenLayers.Feature.Vector quand on recopie les features. Il faut écrire :
Code:

1 2 3 4 5 6 var feature= new OpenLayers.Feature.Vector( this.features[i].geometry.clone(), // on s'autorise quand même à cloner la Géométrie JSON.parse(JSON.stringify(this.features[i].attributes)), JSON.parse(JSON.stringify(this.features[i].style)) ) ;
La différence est qu'on passe trois paramètres au constructeur et non pas un seul objet qui contient 3 propriétés comme on faisait avant.
(Ah, si je tenais celui qui t'avais dit d'écrire ça... :oops:)

Du coup, tu peux supprimer la surcharge de OpenLayers.Geometry.Collection.calculateBounds que je t'avais fait rajouter dans mon dernier message.

Une fois tout cela corrigé, on a enfin la couche qui s'affiche sur le deuxième viewer (:D), mais toujours pas de popups (:aie:).

2. L'affichage des popups dans le deuxième viewer !

Il faut en fait rajouter les options qui vont bien après la création de couche viewer2Layer. En effet, celles-ci ne sont pas intégrées lorsqu'on fait un simple appel au constructeur de OpenLayers.Layer.Vector. Il faut donc écrire le code suivant après lorsqu'on ajoute la couche dans le deuxième viewer :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 viewer2Layer = new OpenLayers.Layer.Vector("Commune sur viewer2", { opacity:1.0, onFeatureInsert: Geoportal.Popup.Anchored.createPopUpForKMLFeature }); viewer2.map.addLayer(viewer2Layer); // popup activation : var xmlDefaults= viewer2.map.getPopupDefaults("KML"); var select= new OpenLayers.Control.SelectFeature(viewer2Layer, xmlDefaults); viewer2.map.addControl(select); select.activate();
Et les popups devraient s'afficher...

05/05/2014, 13h24
cmail

OK! Maintenant, je vais essayer d'adapter cela à mon projet.

Mais avant de me lancer, sur le principe, j'aimerais bien avoir ton sentiment:
J'ai 3 viewers qui gèrent n couches identiques. Donc au lieu de charger 3xn couches avec des délais de réponse en conséquence, j'envisage seulement de télécharger n couches puis de les cloner dans mes deux autres viewers. Vois-tu quelques problèmes à venir?
05/05/2014, 16h51
gcebelieu

Christian,

le fait de cloner les couches te fait économiser le temps de chargement du fichier puis celui de parsing du format KML pour créer les features des couches vecteur.
Dans l'absolu et d'un point de vue intellectuel, c'est mieux.

Dans la pratique, je pense que cela n'améliorera que très légèrement le temps de chargement de la page dans la mesure où ton KML est sur le même serveur que ta page et qu'il n'est pas énorme (environ 125 Ko) comparé aux autres éléments chargés (API JS notamment : environ 2 Mo)...

Le mieux est de tester les deux méthodes pour te faire une idée.

Ce que je peux te dire est que la méthode préconisée dans cette discussion est "saine" : je ne pense pas que tu rencontreras une impossibilité théorique si tu la poursuis.
Pour info, il y a encore un problème de curseur à régler après les corrections que je t'ai fournies ce matin, mais je pense que c'est solvable...