New issue
Advanced search Search tips
Starred by 2 users

Issue metadata

Status: Released
Owner: ----
Closed: Aug 2015



Sign in to add a comment
link

Issue 3366: NewProjectCreatedListeners are called too early.

Reported by jalonso...@gmail.com, May 13 2015

Issue description

*****************************************************************
*****                                                       *****
***** !!!! THIS BUG TRACKER IS FOR GERRIT CODE REVIEW !!!!  *****
*****                                                       *****
***** DO NOT SUBMIT BUGS FOR CHROME, ANDROID, CYANOGENMOD,  *****
***** INTERNAL ISSUES WITH YOUR COMPANY'S GERRIT SETUP, ETC.*****
*****                                                       *****
*****   THOSE ISSUES BELONG IN DIFFERENT ISSUE TRACKERS     *****
*****                                                       *****
*****************************************************************

Affected Version:

What steps will reproduce the problem?
1. Create a plugin that implements NewProjectCreatedListener.
2. Create a new project.
3. Within the plugin, attempt to query Project info for the new project, for example, to find parent project.

What is the expected output? What do you see instead?

If project 'bar' is created with project 'foo' as the parent, expect to see 'foo' as the parent in Project info.

Instead, only ever see 'null' or "All_Projects".

Please provide any additional information below.

It is a _very_ simple change to make this work as expected... a simple move of a few lines.
 
CreateProject.java
15.4 KB View Download

Comment 1 by david.pu...@sonymobile.com, May 14 2015

Status: AwaitingInformation
We cannot accept patches attached to the issue tracker.  If you would like to contribute this fix please complete the license agreement [1] and then push the change for review on gerrit-review.

[1] https://gerrit-review.googlesource.com/#/settings/agreements

Comment 2 by jalonso...@gmail.com, May 14 2015

It wasn't meant as a patch, just supplied so you could diff against the original to see how simple a change it is.  FYI, I have tested this, and it works as expected, with no apparent side-effects.  I suppose I'll look into how to push a patch.

Comment 3 by david.pu...@sonymobile.com, May 19 2015

> I suppose I'll look into how to push a patch.

Thanks.  If it's a simple fix it should be quick to merge.  (I have not looked at the attached patch file).

Comment 4 by david.pu...@sonymobile.com, Aug 1 2015

Comment 5 by edwin.ke...@gmail.com, Aug 4 2015

Project Member
Labels: FixedIn-2.11.3
Status: Submitted

Comment 6 by david.pu...@sonymobile.com, Aug 21 2015

Status: Released

Sign in to add a comment