Cr8-xyz/blender
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Animation: improved error handling when adding/deleting keyframes

Browse Source 
There are two ways in which adding/deleting keyframes can fail, but only
one of those was handled. The other would be shown as a successful
add/delete of -2 keyframes.
This commit is contained in:
Sybren A. Stüvel
2020-03-06 11:36:37 +01:00
parent d8491cb7c6
commit 7d61132807
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
source/blender/editors/animation/keyframing.c
View File

@@ -1845,11 +1845,11 @@ static int insert_key_exec(bContext *C, wmOperator *op)
}
/* report failure or do updates? */
if (num_channels == MODIFYKEY_INVALID_CONTEXT) {
if (num_channels < 0) {
BKE_report(op->reports, RPT_ERROR, "No suitable context info for active keying set");
return OPERATOR_CANCELLED;
}
else if (num_channels) {
else if (num_channels > 0) {
/* if the appropriate properties have been set, make a note that we've inserted something */
if (RNA_boolean_get(op->ptr, "confirm_success")) {
BKE_reportf(op->reports,
@@ -2061,11 +2061,11 @@ static int delete_key_exec(bContext *C, wmOperator *op)
}
/* report failure or do updates? */
if (num_channels == MODIFYKEY_INVALID_CONTEXT) {
if (num_channels < 0) {
BKE_report(op->reports, RPT_ERROR, "No suitable context info for active keying set");
return OPERATOR_CANCELLED;
}
else if (num_channels) {
else if (num_channels > 0) {
/* if the appropriate properties have been set, make a note that we've inserted something */
if (RNA_boolean_get(op->ptr, "confirm_success")) {
BKE_reportf(op->reports,