In the report overview select multiple reports and select one of the
available entries in the context menu opened with right mouse click.
Cherry-picked from branch 4.8, commit <to-be-filled-out>
Adjusted for KF5 by Thomas Baumgart.
BUG:401080
FIXED-IN:4.8.3, 5.0.3
compiled and tested on linux
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
Other than that one thing it looks OK to me, well what else do we expect when the author reviews his own stuff ;-)
| kmymoney/plugins/views/reports/kreportsview.cpp | ||
|---|---|---|
| 615 | Can you join this with the line above? Did I mess this up? | |
| kmymoney/plugins/views/reports/kreportsview.cpp | ||
|---|---|---|
| 615 | Comparing with the original 4.8 related changes https://cgit.kde.org/kmymoney.git/tree/kmymoney/views/kreportsview.cpp?h=4.8&id=bf6278f923ca1307a229c9ca1e8b206a4b82f400#n1100, this is okay. Configuring makes only sense if only one item is selected. | |
| kmymoney/plugins/views/reports/kreportsview.cpp | ||
|---|---|---|
| 615 | Sorry, I just meant to move the open paren onto the line with the if statement. This does not change the logic which indeed is exactly what we need: configuring just makes sense if only a single report is selected. | |