Skip to content

Commit a8dc21e

Browse files
author
Achyut Kumar M
committed
add isUndo to createChangeset to prevent StoredChangeset creation for undo operation
1 parent ce9db73 commit a8dc21e

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

GoInfoGame/GoInfoGame/DataBase/DatabaseConnector.swift

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -427,7 +427,9 @@ class DatabaseConnector {
427427
- parameter tags [String:String] tags changed with this
428428
- Returns: An instance of `StoredChangeset`
429429
*/
430-
func createChangeset(id:String, type: StoredElementEnum, tags:[String:String]) -> StoredChangeset? {
430+
func createChangeset(id:String, type: StoredElementEnum, tags:[String:String], isUndo: Bool) -> StoredChangeset? {
431+
if isUndo { return nil }
432+
431433
let storedChangeset = StoredChangeset()
432434
storedChangeset.elementId = id
433435
storedChangeset.elementType = type

GoInfoGame/GoInfoGame/quests/QuestProtocols.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,11 +39,11 @@ class QuestBase {
3939

4040
// Add a custom implementation
4141

42-
public func updateTags(id: Int64, tags:[String:String], type: ElementType){
42+
public func updateTags(id: Int64, tags:[String:String], type: ElementType, isUndo: Bool = false) {
4343

4444
if MapUndoManager.shared.updateTagsHandler == nil {
4545
MapUndoManager.shared.updateTagsHandler = { [weak self] id, tags, type in
46-
self?.updateTags(id: id, tags: tags, type: type)
46+
self?.updateTags(id: id, tags: tags, type: type, isUndo: true)
4747
}
4848
}
4949

@@ -52,7 +52,7 @@ class QuestBase {
5252
let storedElementType: StoredElementEnum = type == .way ? .way : .node
5353
let storedId = String(id)
5454
// Create a changeset
55-
_ = DatabaseConnector.shared.createChangeset(id: storedId, type: storedElementType, tags: tags)
55+
_ = DatabaseConnector.shared.createChangeset(id: storedId, type: storedElementType, tags: tags, isUndo: isUndo)
5656
switch (storedElementType){
5757
case .way:
5858
elementSubmittingToPOSM = .way

0 commit comments

Comments
 (0)