Minyaa Suite

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: None
• Fix Version/s: 3.2.1
• Component/s: None
• Security Level: Public
• Labels:

  None

If CustomfieldManager is called by any Plugin during the start of JIRA (here GreenHopper, but it may true for any other plugin), then the CustomfieldManager tries to load any defined customfield.
If one these customfield has dependencies on component provided by a plugin, this plugin may be not yet loaded. In such case, a org.picocontainer.defaults.UnsatisfiableDependenciesException is raised, JIRA is locked and is not available.

Classically, a Customfield looks like :

public class MyCFType {
   protected final OneJIRAComponent oneJIRAComponent;
   protected final OtherJIRAComponent otherJIRAComponent;
   protected final MyPluginComponent myPluginComponent;
	
   public MyCFType(OneJIRAComponent oneJIRAComponent, OtherJIRAComponent otherJIRAComponent, MyPluginComponent myPluginComponent) {
      this.oneJIRAComponent = oneJIRAComponent;
      this.otherJIRAComponent = otherJIRAComponent;
      this.myPluginComponent = myPluginComponent;
   }
   
   public anyMethod() {
      oneJIRAComponent.doSomething();
      otherJIRAComponent.doSomething();
      myPluginComponent.doSomething();
   }
}

To avoid the current issue, it would be better to do :

public class MyCFType {
   protected final OneJIRAComponent oneJIRAComponent;
   protected final OtherJIRAComponent otherJIRAComponent;
   protected MyPluginComponent myPluginComponent;

   public MyCFType(OneJIRAComponent oneJIRAComponent, OtherJIRAComponent otherJIRAComponent) {
      this.oneJIRAComponent = oneJIRAComponent;
      this.otherJIRAComponent = otherJIRAComponent;
   }
   
   public anyMethod() {
      oneJIRAComponent.doSomething();
      otherJIRAComponent.doSomething();
      getMyPluginComponent().doSomething();
   }
   
   public MyPluginComponent getMyPluginComponent() {
      if (myPluginComponent == null) {
         myPluginComponent = (MyPluginComponent)ComponentManager.getComponentInstanceOfType(MyPluginComponent.class);
      }
      return myPluginComponent;
   }
}

• Assignee:

  Vincent Thoulé

  Reporter:

  Vincent Thoulé

• Created:

  16/Sep/11 7:49 AM

  Updated:

  18/Sep/11 4:51 AM

  Resolved:

  18/Sep/11 4:51 AM