[Bug 27378] New: New distributions submitted in test reports are added even if the admin deletes them when processing

http://bugs.winehq.org/show_bug.cgi?id=27378

Summary: New distributions submitted in test reports are added
                    even if the admin deletes them when processing
           Product: WineHQ Apps Database
           Version: unspecified
          Platform: x86
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: appdb-unknown
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

Users have a penchant for submitting "new" distributions that already have
entries in the AppDB (bug 24497). The form admins and maintainers see when
processing test reports notes when the user has added a new distribution, and
states that it will be added to the database unless the admin/maintainer
selects an existing one from the list. However, even if I do select a
distribution from the list and delete the "new" distribution from the test form
when processing it, the "new" distribution is still added to the database and
has to be manually removed afterwards.

[Bug 27378] New distributions submitted in test reports are added even if the admin deletes them when processing

http://bugs.winehq.org/show_bug.cgi?id=27378

Ken Sharp  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

[Bug 27378] New distributions submitted in test reports are added even if the admin deletes them when processing

http://bugs.winehq.org/show_bug.cgi?id=27378

Ken Sharp  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |download, source

--- Comment #1 from Ken Sharp  2012-12-24 07:32:21 CST ---
"The Distribution was successfully added into the database."

Still present.

[Bug 27378] New distributions submitted in test reports are added even if the admin deletes them when processing

https://bugs.winehq.org/show_bug.cgi?id=27378

--- Comment #2 from Rosanne DiMesio  ---
Still present.

[Bug 27378] New distributions submitted in test reports are added even if the admin deletes them when processing

https://bugs.winehq.org/show_bug.cgi?id=27378

--- Comment #3 from Rosanne DiMesio  ---
Still an issue, and the problem is even worse than originally described: the
distribution is not changed in the test report, so it can't simply be deleted;
the admin has to move the test report to another distribution (or delete it)
first.

[Bug 27378] New distributions submitted in test reports are added even if the admin deletes them when processing

https://bugs.winehq.org/show_bug.cgi?id=27378

--- Comment #4 from Rosanne DiMesio  ---
Commit c3635c89ab31de3a272aab97bda350d65f224837 should fix the additional
problem mentioned in comment 3, though not the original problem. New
distributions are still incorrectly added to the database, but without any
tests attached, and test reports are added with the correct distribution
information. Not yet fixed, but IMO somewhat easier to deal with.

[Bug 27378] New distributions submitted in test reports are added even if the admin deletes them when processing

https://bugs.winehq.org/show_bug.cgi?id=27378

Rosanne DiMesio  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED
      Fixed by SHA1|                            |8cf9bd53fc0f5251bdba75ae882
                   |                            |15fa1c0da8880

--- Comment #5 from Rosanne DiMesio  ---
Marking fixed.

[Bug 27378] New distributions submitted in test reports are added even if the admin deletes them when processing

https://bugs.winehq.org/show_bug.cgi?id=27378

Rosanne DiMesio  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #6 from Rosanne DiMesio  ---
Closing fixed AppDB bug.