Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Browse files
Browse the repository at this point in the history
[BUGFIX] The way layer's shortname is write in QGIS project and used …
…in QGIS Server
- Loading branch information
Showing
2 changed files
with
26 additions
and
15 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
849a756
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rldhont Thanks for the fix.
Is there no risk involved in omitting these nodes when writing to the project? E.g. that a project saved with 2.14.2 will be incompatible with 2.14.1?
It would be nice if you could include some rationale for changes in the commit message, I'm still not sure why these changes were required.
849a756
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the element shortname, title or abstract are only for QGIS Server. No need to add empty element in QGIS Project.
The important part of the fix is:
If QGIS Server found an empty shortname element, it used the empty string has layername. Now it used layername is shortname is empty.