#29 importmodule changes database on failed album import

closed-fixed
Shanks
7
2009-04-10
2009-04-10
KWUR Tech
No

this should not happen - a failed album import for any reason should not alter the db. instead, the track paths get changed to the active archive directory even though the pending status is unchanged

Discussion

  • KWUR Tech
    KWUR Tech
    2009-04-10

    • assigned_to: nobody --> tantricengineer
     
  • Shanks
    Shanks
    2009-04-10

    restructured approvePending(id) and move(id) methods to degrade in the event of a failed import. album approval now happens in the following way:
    1) try to move the album -> if this fails, we return error
    2) if we successfully move album -> alter database to set album pending status to 0 and alter track paths

     
  • Shanks
    Shanks
    2009-04-10

    • status: open --> closed-fixed