Mid-Air Collision checking for Bug.update()

NEW
Unassigned

Status

()

Bugzilla
WebService
--
enhancement
10 years ago
4 years ago

People

(Reporter: Max Kanat-Alexander, Unassigned)

Tracking

Dependency tree / graph

Details

Attachments

(1 attachment)

(Reporter)

Description

10 years ago
Implement an optional parameter for the Bug.update() WebService call, called check_collision, which takes a date.

If this is specified and there's been a mid-air collision since that time, we will return an item called 'collision' that is a hash representing the changes since that time and who made them.
(Reporter)

Comment 1

10 years ago
As I started to implement this, I realized that it was entirely handled by bug 424079--that is, once we have get_activity, we can actually just call it directly from Bug.update() when there's a midair collision. So I attached the code that I had for this to bug 424079 for nelwahar to add in to his code.
Depends on: 424079

Comment 2

6 years ago
We are going to branch for Bugzilla 4.4 next week and this bug is either too invasive to be accepted for 4.4 at this point or shows no recent activity. The target milestone is reset and will be set again *only* when a patch is attached and approved.

I ask the assignee to reassign the bug to the default assignee if you don't plan to work on this bug in the near future, to make it clearer which bugs should be fixed by someone else.
Target Milestone: Bugzilla 4.4 → ---

Updated

6 years ago
Assignee: mkanat → webservice

Comment 3

5 years ago
Created attachment 816342 [details] [diff] [review]
432910_V1.patch

I developed this for the Mylyn Bugzilla REST API connector.
Attachment #816342 - Flags: review?(dkl)
Comment on attachment 816342 [details] [diff] [review]
432910_V1.patch

Review of attachment 816342 [details] [diff] [review]:
-----------------------------------------------------------------

Sorry for taking so long to get to this :( Unfortunately there is some bit rot and needs to be updated. Let me know if you no longer want to work on this as well and I can take it over.

dkl

::: Bugzilla/WebService/Bug.pm
@@ +3868,4 @@
>  
>  =item REST API call added Bugzilla B<5.0>.
>  
> +=item Add Returnfield "collision" in Bugzilla B<5.0>.

s/collision/collisions/
Attachment #816342 - Flags: review?(dkl) → review-

Comment 5

4 years ago
(In reply to David Lawrence [:dkl] from comment #4)
> Comment on attachment 816342 [details] [diff] [review]
> 432910_V1.patch
> 
> Review of attachment 816342 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Sorry for taking so long to get to this :( Unfortunately there is some bit
> rot and needs to be updated. Let me know if you no longer want to work on
> this as well and I can take it over.
> 
> dkl

It would be great if you continue with this.

Frank
You need to log in before you can comment on or make changes to this bug.