New issue
Advanced search Search tips
Starred by 2 users

Issue metadata

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



Sign in to add a comment

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
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
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.
> 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).
Status: ChangeUnderReview
https://gerrit-review.googlesource.com/#/c/70001/
Project Member

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

Labels: FixedIn-2.11.3
Status: Submitted
Status: Released

Sign in to add a comment