Issue 3366: NewProjectCreatedListeners are called too early.
Status:  Released
Owner: ----
Closed:  Aug 4
Reported by JalonSo...@gmail.com, May 13, 2015
*****************************************************************
*****                                                       *****
***** !!!! 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
May 14, 2015
Project Member #1 david.pu...@sonymobile.com
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
Status: AwaitingInformation
May 14, 2015
#2 JalonSo...@gmail.com
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.
May 18, 2015
Project Member #3 david.pu...@sonymobile.com
> 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).
Aug 1, 2015
Project Member #4 david.pu...@sonymobile.com
https://gerrit-review.googlesource.com/#/c/70001/
Status: ChangeUnderReview
Aug 4, 2015
Project Member #5 edwin.ke...@gmail.com
(No comment was entered for this change.)
Status: Submitted
Labels: FixedIn-2.11.3
Aug 20, 2015
Project Member #6 david.pu...@sonymobile.com
(No comment was entered for this change.)
Status: Released