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

Regression with expressions in Composer #29288

Closed
qgib opened this issue Mar 4, 2019 · 5 comments
Closed

Regression with expressions in Composer #29288

qgib opened this issue Mar 4, 2019 · 5 comments
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! High Priority Print Layouts Related to QGIS Print Layouts, Atlas or Reporting frameworks Regression Something which used to work, but doesn't anymore

Comments

@qgib
Copy link
Contributor

qgib commented Mar 4, 2019

Author Name: Pedro Venâncio (Pedro Venâncio)
Original Redmine Issue: 21471
Affected QGIS version: 3.4.5
Redmine category:map_composer/printing


I updated QGIS 3.4.5, 3.6.0 and 3.7.0 today to 3.4.5-3 (code revision 8ba90fc), 3.6.0-4 (code revision ec30c48) and 3.7.0-4 (code revision 34a0650) with OSGeo4W, and the labels with expressions are not rendered as expected, when are used more than one line.

For instance,

[% CASE WHEN month($now) = 3 THEN 'March' END || ' ' || year($now) %]

renders ok

March 2019

but

[% 
    CASE WHEN month($now) = 3 THEN 'March' END || ' ' || year($now) 
%]

renders

[% 
    CASE WHEN month($now) = 3 THEN 'March' END || ' ' || year($now) 
%]

I'm sure this is a regression, because it was working fine before today's update. Last week this was working fine.

@qgib
Copy link
Contributor Author

qgib commented Mar 4, 2019

Author Name: Giovanni Manghi (@gioman)


  • subject was changed from Problem with expressions in Composer to Regression with expressions in Composer

@qgib
Copy link
Contributor Author

qgib commented Mar 4, 2019

Author Name: Nyall Dawson (@nyalldawson)


Applied in changeset daeb3d8.


  • done_ratio was changed from 0 to 100
  • status_id was changed from Open to Closed

@qgib
Copy link
Contributor Author

qgib commented Mar 6, 2019

Author Name: Pedro Venâncio (Pedro Venâncio)


Hi Nyall,

Thank you very much for the fix!

But this also needs to be backported to release-3_4 (LTR).

Thanks!


  • status_id was changed from Closed to Reopened

@qgib
Copy link
Contributor Author

qgib commented Mar 6, 2019

Author Name: Giovanni Manghi (@gioman)


Hi Pedro, I think we can keep this closed as the bug is fixed, for heads up about backporting I think is better to leave messages in the PR on GitHub.

@qgib
Copy link
Contributor Author

qgib commented Mar 6, 2019

Author Name: Nyall Dawson (@nyalldawson)


There's already a 3.4 backported pr, it just hasn't been merged yet


  • status_id was changed from Reopened to Closed

@qgib qgib closed this as completed Mar 6, 2019
@qgib qgib added Bug Either a bug report, or a bug fix. Let's hope for the latter! High Priority Print Layouts Related to QGIS Print Layouts, Atlas or Reporting frameworks Regression Something which used to work, but doesn't anymore labels May 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! High Priority Print Layouts Related to QGIS Print Layouts, Atlas or Reporting frameworks Regression Something which used to work, but doesn't anymore
Projects
None yet
Development

No branches or pull requests

1 participant