Bug report #10654

"Label features multiple times" doesn't work if "merge connected lines to avoid duplicate labels" is active

Added by luca76 - over 10 years ago. Updated over 10 years ago.

Status:Closed
Priority:Severe/Regression
Assignee:Larry Shaffer
Category:Labelling
Affected QGIS version:master Regression?:No
Operating System: Easy fix?:No
Pull Request or Patch supplied:No Resolution:fixed/implemented
Crashes QGIS or corrupts data:No Copied to github as #:19053

Description

Hi,

I tested the new qgis 2.4 feature "Label features multiple times" on the attached layer, but it doesn't work.

To test it:

1) Label with field: "desvia"
2) enable "merge connected lines to avoid duplicate labels" option
3) put any value on "repeat"
4) no label is repeated (see the road "VIA GIUSEPPE VERDI", for example, that is a long road)

History

#1 Updated by luca76 - over 10 years ago

  • Priority changed from Normal to Severe/Regression

Putting it to Blocker since it's a new feature of 2.4 that is not working

#2 Updated by Giovanni Manghi over 10 years ago

  • Subject changed from "Label features multiple times" doesn't work to "Label features multiple times" doesn't work if "merge connected lines to avoid duplicate labels" is active
  • Category set to Labelling
  • Status changed from Open to Feedback
  • Affected QGIS version changed from 2.2.0 to master

it works if "merge connected lines to avoid duplicate labels" is not active, make kind of sense to me(?).

#3 Updated by luca76 - over 10 years ago

the point is that in the "Visual changelog for 2.4" http://changelog.linfiniti.com/qgis/version/2.4.0/

it states that:

"This also works in combination with the "merge connected lines to avoid duplicate labels" option."

#4 Updated by Giovanni Manghi over 10 years ago

  • Target version set to Version 2.4

oooh right :)

#5 Updated by Larry Shaffer over 10 years ago

  • Assignee set to Larry Shaffer

Hi,

This is addressed here: https://github.com/qgis/QGIS/pull/1395

Looking into whether it can be pushed (after reviewing) before 2.4 release.

#6 Updated by Giovanni Manghi over 10 years ago

  • Status changed from Feedback to Open

#7 Updated by Andreas Neumann over 10 years ago

I would very much appreciate if this patch could be applied. We paid for this feature - so we have an interest that it works properly.

Thanks for reviewing!
Andreas

#8 Updated by Larry Shaffer over 10 years ago

Andreas Neumann wrote:

I would very much appreciate if this patch could be applied. We paid for this feature - so we have an interest that it works properly.

I have not heard a 'no' from my question as to whether the pull request can be pushed for 2.4, so I will review it tonight with intentions of pushing it to current master.

#9 Updated by Giovanni Manghi over 10 years ago

  • Resolution set to fixed/implemented
  • Status changed from Open to Closed

the patch has been merged.

Also available in: Atom PDF