Need to investigate, but on a project the order seems to reverse

CommentFileSizeAuthor
#12 3070987-12.patch2.48 KBswentel
#11 3070987-11.patch2.16 KBswentel
#10 3070987-10.patch1.38 KBswentel
#4 3070987-4.patch638 bytesswentel

Comments

swentel created an issue. See original summary.

swentel’s picture

swentel commented

Can't reproduce immediately on a vanilla install, digging further.

swentel’s picture

swentel commented

Actually I can, need to add 3 blocks and then the weight goes random :)

swentel’s picture

swentel commented
Status: Active » Needs review
StatusFileSize
new 3070987-4.patch638 bytes
swentel’s picture

swentel commented
Status: Needs review » Fixed

  • swentel committed e39d4aa on 8.x-2.x 
    Issue #3070987 by swentel: Weight of compenents doesn't seem to be...
swentel’s picture

swentel commented
Title: Weight of compenents doesn't seem to be respected when copying » Weight of components doesn't seem to be respected when copying

  • swentel committed e92ed44 on 8.x-2.x 
    Revert "Issue #3070987 by swentel: Weight of compenents doesn't seem to...
swentel’s picture

swentel commented
Status: Fixed » Active

Need to reopen. It doesn't work in all cases, throwing exceptions. I don't get why setComponent() is set to be protected, because that's the method I really need :/

swentel’s picture

swentel commented
Priority: Normal » Major
Status: Active » Needs review
StatusFileSize
new 3070987-10.patch1.38 KB

Testing different approach

swentel’s picture

swentel commented
StatusFileSize
new 3070987-11.patch2.16 KB

Better approach, tested manually. This also makes sure in case an inline block is not copied, the weights are following nicely.

swentel’s picture

swentel commented
StatusFileSize
new 3070987-12.patch2.48 KB

Cleanup - no more removal needed at the bottom.

  • swentel committed 7808c8f on 8.x-2.x 
    Issue #3070987 by swentel: Weight of components doesn't seem to be...
swentel’s picture

swentel commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.