Opened 6 years ago

Closed 6 years ago

#1857 closed bug (fixed)

deleteMTrad function uses MakeRevision on wrong data

Reported by: lantti Owned by: lantti
Priority: major Milestone: 1.3
Component: BW Database Keywords: translation, revision, database
Cc:

Description

A call to the MakeRevision? function before deleting a translation for a text is done using an IdTrad? instead of a row id. This results in writing a revision entry for a completely random piece of database data.

Change History (5)

comment:1 Changed 6 years ago by shevek

Does this need to be fixed to get #1850 fixed?

comment:2 Changed 6 years ago by lantti

No, this breaks only the revision log (previousversion table on db) and I don't see that data being used for anything except admin stuff. I just stumbled on it while browsing the db for clues for #1850.

comment:3 Changed 6 years ago by shevek

Fixed along with #1850.

comment:4 Changed 6 years ago by shevek

  • Milestone changed from unassigned to 1.3

comment:5 Changed 6 years ago by jsfan

  • Resolution set to fixed
  • Status changed from new to closed

Tested and works. Closed.

Note: See TracTickets for help on using tickets.