Skip to content

Commit f0667c1

Browse files
committed
Changed Rollback to Virtual
All methods in `TopicRepositoryBase` should allow optional overriding in concrete implementations, if necessary—even though `Rollback` may not generally need it, due to its nature. Fixed formatting for `args` while I was at it (since it's not near other variable declarations, the alignment isn't necessary or consistent).
1 parent c8d6011 commit f0667c1

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

Ignia.Topics/Repositories/TopicRepositoryBase.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,7 @@ public static Topic Load(XmlNode node, ImportStrategy importStrategy = ImportStr
152152
/// exception="T:System.ArgumentNullException">
153153
/// !VersionHistory.Contains(version)
154154
/// </requires>
155-
public void Rollback(Topic topic, DateTime version) {
155+
public virtual void Rollback(Topic topic, DateTime version) {
156156

157157
/*------------------------------------------------------------------------------------------------------------------------
158158
| Retrieve topic from database
@@ -238,7 +238,7 @@ public virtual int Save(Topic topic, bool isRecursive = false, bool isDraft = fa
238238
| Trigger event
239239
\-----------------------------------------------------------------------------------------------------------------------*/
240240
if (topic.OriginalKey != null && topic.OriginalKey != topic.Key) {
241-
var args = new RenameEventArgs(topic);
241+
var args = new RenameEventArgs(topic);
242242
RenameEvent?.Invoke(this, args);
243243
}
244244

0 commit comments

Comments
 (0)