Discussion :

[cmail]

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?

[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

[cmail]

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

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

[gcebelieu]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

par

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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).

[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????

[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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

par

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
attributes : JSON.parse(JSON.stringify(this.features[i].attributes)),
style: JSON.parse(JSON.stringify(this.features[i].style))

[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...

[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 :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
geometry: JSON.parse(JSON.stringify(this.features[i].geometry));

[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...

[gcebelieu]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
geometry: this.features[i].geometry.clone(),

2. tu rajoutes ce code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[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....

[cmail]

Et mes clones?

[gcebelieu]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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... )

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 (), mais toujours pas de popups ().

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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...

[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?

[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...