Copy-paste a multimeasure rest on another one causes corruption after undo

Fixed in branch 2.1, commit bef0b6e12e

fix #173381: Copy-paste a multimeasure rest on another one causes corruption after undo

Leaving it open for 3.0 backport.

Cannot reproduce the case. All rests are on the correct places.

The other issue is that "M" shortcut doesn't work on my machine with MuseScoreNightly-2018-04-05-2004-master-82e81eb.dmg (MacOS).

After rebooting Mac, I can reproduce the issue in the main score, not in the part view. Also, after the corruption happens, closing the scoreleads to the crash.

Works correct in the part and main score views on latest nightly.

Again appear.
1. Empty score
2. Create 4 quarter notes in the first measure
3. Switch on MM rests
4. Select 1st measure
5. Press "R"
6. Select 1st measure again
7. Press "R"
8. Press "R" again
Result: last measure becomes weird
9. Undo until switching MM rests off happens
Result: Score is corrupted.

Reopen then?

I'll add a bit to this discussion.
First, the discussed problem can lead not only to score corruption but also to a crash. To reproduce it do the following steps:
1) Create a score like it is described in steps 1—3 in the comment by Anatoly-os.
2) Select only three notes in range instead of the whole measure.
3) Press R two times.
Musescore will then crash.

Second, I am not really sure which is a good way to fix this. Score::pasteStaff function range-selects the pasted content after it has been actually pasted. However:
1) It is not possible to correctly determine what to select by finding the bounding segments by tick values (as it is currently done) since there are some types of segments which have zero tick length and still can be selected (e.g. Clefs).
2) Multi-measure rests are rebuilt a bit later during the layout so any selection that intersects the previous multi-measure rest range will behave unreliably (MM-rest layout process involves some segments deletion, changes in their structure etc.). This can lead to the selection being in invalid state (or at least in a state which was not intended and which is impossible to reproduce via a user interface), and this is probably the reason for the observed peculiarities in selection behavior such as disappearing selection when the pasted area is inside the previous location of the MM-rest and the described crashes.

So it is not currently clear for me how to ensure that the selection state is correct after all those operations without introducing changes to the code which are larger than it could be desired.

See https://github.com/musescore/MuseScore/pull/4133, https://github.com/musescore/MuseScore/pull/4134

Fixed in branch master, commit fa2c795d07

fix #173381, part 1: handle selection after MMRest reconstruction

Make selection made after paste operation be actually updated after
re-layout and, hence, after reconstructing multi-measure rests.
This is needed as reconstructing MMRests changes segments structure
of the score which leads selection to be in invalid state.
This commit adds also some tweaks to selection boundaries to work
around the fixed bug (which would not work alone without the
previously described changes).

Fixed in branch master, commit d744cf843b

Merge pull request #4133 from dmitrio95/173381-mmrest-repeat

fix #173381, part 1: handle selection after MMRest reconstruction

Fixed in branch master, commit 30332b1aba

fix #173381, part 2: assign correct duration to the pasted rests

The added condition is needed as TDuration::fraction() returns
invalid fraction (0/0) if duration type is whole-measure.
Still that behavior for TDuration can be considered correct as
TDuration object does not have any information on its context,
including measure's length.

Fixed in branch master, commit 269f8504b7

Merge pull request #4134 from dmitrio95/173381-mmrest-copy

fix #173381, part 2: assign correct duration to the pasted rests

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