addWMS funktioniert nicht mehr

[Stolz]

Hallo zusammen,

addWMS funktioniert nicht mehr. Es wird kein JavaScript-Fehler generiert.

In \src\modules\tools\addWMS\components\AddWMS.vue steigt der Import aus in

Radio.trigger("Parser", "addLayer", object.Title, this.getParsedTitle(object.Title), parentId, level, object.Name, this.wmsUrl, this.version);

In 2.9 funktioniert das alles noch. Hat das evtl. etwas damit zu tun das danach “The Id in config.json for every layer could be in an object format” möglich ist?

Danke&VG

Dietmar Stolz

[Dominik Fröhlich]

Hallo Herr Stolz,

ich habe das Problem mal analysiert. Die Ursache sind nicht funktionierende defaults in der /modules/core/configLoader/parser.js::364.

Das addWMS Tool funktioniert, wenn man in /src/modules/tools/addWMS/components/AddWMS.vue::235

Radio.trigger("Parser", "addLayer", object.Title, this.getParsedTitle(object.Title), parentId, level, object.Name, this.wmsUrl, this.version, true, false, false);

statt

Radio.trigger("Parser", "addLayer", object.Title, this.getParsedTitle(object.Title), parentId, level, object.Name, this.wmsUrl, this.version);

verwendet (also die defaults explizit mit übergibt).

‌

Grüße aus Freiburg!

Dominik Fröhlich

[Stolz]

Hallo Herr Fröhlich,

Ihr Name ist Programm, tausend Dank, damit funktioniert’s.

VG

Dietmar Stolz

[Christopher Reder]

Hallo Herr Stolz, sie können die Lösung gerne als pull request ans Masterportal stellen, damit der Fehler auch in den nachfolgenden Versionen gefixt ist. Gerne auch mit entsprechenden Tests um ein erneutes Auftreten des Problems zu verhindern.

Mit freundlichen Grüßen Christopher Reder

[Dominik Fröhlich]

Hallo Herr Reder,

der Fix war von mir. Sobald mein fork läuft kann ich auch dazu einen Pull request stellen. Ich will aber betonen, dass das nur ein Hotfix ist. Der eigentliche Bug ist, dass der default in der parser.js nicht tut.

Gruß

Dominik Fröhlich

[Pascal Röhling]

Fix parameter defaults of addLayer in parser.js

The function “addLayer” now accepts three optional arguments transparent (boolean), isSelected (boolean) and time (object or boolean) instead of one object containing these three.

Fixes: Issue #640

→ <<cset 539372400695>>