Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport release-3_6] of SRCHEIGHT/SRCWIDTH for GetLegendGraphic request #9727

Merged
merged 3 commits into from
Apr 11, 2019

Conversation

signedav
Copy link
Contributor

@signedav signedav commented Apr 5, 2019

It takes these values as map size in case of GetLegendGraphics Request and still HEIGHT and WIDTH if not a GetLegendGraphics Request.

Backport of #9545

it takes these values as map size in case of GetLegendGraphics Request and still HEIGHT and WIDTH if not a GetLegendGraphics Request

cherry-picked commits 4c667ad e184772 6722ad5 61a89af$
@signedav
Copy link
Contributor Author

signedav commented Apr 7, 2019

I fixed the comment @Gustry
Would you like to review it and this #9726 as well?

@Gustry
Copy link
Contributor

Gustry commented Apr 11, 2019

I'm not a C++ reviewer, I can't comment on that sorry ;-)
But I saw you still have some 3.8 in this PR, I don't know what you should do.

@mbernasocchi mbernasocchi requested review from pblottiere and elpaso and removed request for pblottiere and elpaso April 11, 2019 13:44
/**
* Returns SRCWIDTH parameter or an empty string if not defined.
* \returns srcWidth parameter
* \since QGIS 3.8
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3.4

* converted.
* \returns srcWidth parameter
* \throws QgsBadRequestException
* \since QGIS 3.8
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3.4

/**
* Returns SRCHEIGHT parameter or an empty string if not defined.
* \returns srcHeight parameter
* \since QGIS 3.8
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3.4

* converted.
* \returns srcHeight parameter
* \throws QgsBadRequestException
* \since QGIS 3.8
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3.4

@pblottiere pblottiere merged commit 717501d into qgis:release-3_6 Apr 11, 2019
@pblottiere
Copy link
Member

Thanks @signedav 👍!

@signedav
Copy link
Contributor Author

Thanks to you as well @pblottiere

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants