fix: remove obsolete properties & fix NRE

This commit is contained in:
ZavaruKitsu
2022-10-08 20:37:35 +03:00
parent d566cf858e
commit 8d02b54b05
9 changed files with 28 additions and 45 deletions
+8 -1
View File
@@ -33,7 +33,7 @@ public class MessageResult : ResultBase
/// <summary>
/// The message id
/// </summary>
public new int MessageId =>
public override int MessageId =>
UpdateData?.Message?.MessageId
?? Message?.MessageId
?? UpdateData?.CallbackQuery?.Message?.MessageId
@@ -45,6 +45,13 @@ public class MessageResult : ResultBase
public MessageType MessageType => Message?.Type ?? MessageType.Unknown;
public override Message Message =>
UpdateData?.Message
?? UpdateData?.EditedMessage
?? UpdateData?.ChannelPost
?? UpdateData?.EditedChannelPost
?? UpdateData?.CallbackQuery?.Message;
/// <summary>
/// Is this an action ? (i.e. button click)
/// </summary>
+2 -2
View File
@@ -12,7 +12,7 @@ public class ResultBase : EventArgs
public virtual long DeviceId { get; set; }
public int MessageId => Message.MessageId;
public virtual int MessageId => Message.MessageId;
public virtual Message Message { get; set; }
@@ -42,4 +42,4 @@ public class ResultBase : EventArgs
{
}
}
}
}
+1 -10
View File
@@ -55,15 +55,6 @@ public class ButtonGrid : ControlBase
public string ConfirmationText { get; set; } = "";
/// <summary>
/// </summary>
[Obsolete("This property is obsolete. Please use the DataSource property instead.")]
public ButtonForm ButtonsForm
{
get => DataSource.ButtonForm;
set => DataSource = new ButtonFormDataSource(value);
}
/// <summary>
/// Data source of the items.
/// </summary>
@@ -746,4 +737,4 @@ public class ButtonGrid : ControlBase
break;
}
}
}
}
@@ -55,13 +55,6 @@ public class CheckedButtonList : ControlBase
public string ConfirmationText { get; set; } = "";
[Obsolete("This property is obsolete. Please use the DataSource property instead.")]
public ButtonForm ButtonsForm
{
get => DataSource.ButtonForm;
set => DataSource = new ButtonFormDataSource(value);
}
/// <summary>
/// Data source of the items.
/// </summary>
@@ -277,7 +270,7 @@ public class CheckedButtonList : ControlBase
foreach (var c in CheckedRows)
{
lst.Add(ButtonsForm[c][0]);
lst.Add(DataSource.ButtonForm[c][0]);
}
return lst;
@@ -418,7 +411,7 @@ public class CheckedButtonList : ControlBase
Updated();
await OnCheckedChanged(new CheckedChangedEventArgs(ButtonsForm[index], index, false));
await OnCheckedChanged(new CheckedChangedEventArgs(DataSource.ButtonForm[index], index, false));
}
else if (result.MessageText.EndsWith(UncheckedIconLabel))
{
@@ -435,7 +428,7 @@ public class CheckedButtonList : ControlBase
Updated();
await OnCheckedChanged(new CheckedChangedEventArgs(ButtonsForm[index], index, true));
await OnCheckedChanged(new CheckedChangedEventArgs(DataSource.ButtonForm[index], index, true));
}
//else if (this.EnableSearch)
//{
@@ -589,7 +582,7 @@ public class CheckedButtonList : ControlBase
Updated();
await OnCheckedChanged(new CheckedChangedEventArgs(ButtonsForm[index], index, true));
await OnCheckedChanged(new CheckedChangedEventArgs(DataSource.ButtonForm[index], index, true));
}
break;
@@ -604,7 +597,7 @@ public class CheckedButtonList : ControlBase
Updated();
await OnCheckedChanged(new CheckedChangedEventArgs(ButtonsForm[index], index, false));
await OnCheckedChanged(new CheckedChangedEventArgs(DataSource.ButtonForm[index], index, false));
}
break;
@@ -63,13 +63,6 @@ public class TaggedButtonGrid : MultiView
public string ConfirmationText { get; set; }
[Obsolete("This property is obsolete. Please use the DataSource property instead.")]
public ButtonForm ButtonsForm
{
get => DataSource.ButtonForm;
set => DataSource = new ButtonFormDataSource(value);
}
/// <summary>
/// Data source of the items.
/// </summary>
@@ -1017,4 +1010,4 @@ public class TaggedButtonGrid : MultiView
}
#endregion
}
}