Return-Path: Delivered-To: apmail-felix-dev-archive@www.apache.org Received: (qmail 79045 invoked from network); 13 Jun 2008 22:16:36 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 13 Jun 2008 22:16:36 -0000 Received: (qmail 83636 invoked by uid 500); 13 Jun 2008 22:16:38 -0000 Delivered-To: apmail-felix-dev-archive@felix.apache.org Received: (qmail 83599 invoked by uid 500); 13 Jun 2008 22:16:38 -0000 Mailing-List: contact dev-help@felix.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@felix.apache.org Delivered-To: mailing list dev@felix.apache.org Received: (qmail 83587 invoked by uid 99); 13 Jun 2008 22:16:38 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 13 Jun 2008 15:16:38 -0700 X-ASF-Spam-Status: No, hits=-2000.0 required=10.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.140] (HELO brutus.apache.org) (140.211.11.140) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 13 Jun 2008 22:15:57 +0000 Received: from brutus (localhost [127.0.0.1]) by brutus.apache.org (Postfix) with ESMTP id 046E9234C139 for ; Fri, 13 Jun 2008 15:15:45 -0700 (PDT) Message-ID: <1263776522.1213395345003.JavaMail.jira@brutus> Date: Fri, 13 Jun 2008 15:15:45 -0700 (PDT) From: "Dieter Wimberger (JIRA)" To: dev@felix.apache.org Subject: [jira] Commented: (FELIX-587) org.apache.felix.webconsole.internal.compendium.AjaxConfigManagerAction.configForm(): Configuration instance for a ManagedServiceFactory will cause Exception In-Reply-To: <2055524294.1212299204987.JavaMail.jira@brutus> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org [ https://issues.apache.org/jira/browse/FELIX-587?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12604990#action_12604990 ] Dieter Wimberger commented on FELIX-587: ---------------------------------------- I have checked out a fresh working copy. Creation is possible, but it does not seem to work properly, because it reports: This configuration has no associated description Here are the values returned from the Ajax call: {"pid":"net.wimpi.telnetd.service.TelnetListenerServiceFactory-1213394666652-2","isFactory":false,"title" :"net.wimpi.telnetd.service.TelnetListenerServiceFactory-1213394666652-2","description":"Please enter configuration properties for this configuration in the field below. This configuration has no associated description","propertylist":"properties","properties":{}} There is a corresponding metatype description. The way you approached to fix the chain of issues I reported (sorry for the messed up patch), it occurs difficult to me to track them properly. Also, I have a question for you: Do you actually have a test case? If yes, which one is it? Regards, Dieter > org.apache.felix.webconsole.internal.compendium.AjaxConfigManagerAction.configForm(): Configuration instance for a ManagedServiceFactory will cause Exception > ------------------------------------------------------------------------------------------------------------------------------------------------------------- > > Key: FELIX-587 > URL: https://issues.apache.org/jira/browse/FELIX-587 > Project: Felix > Issue Type: Bug > Components: Web Console > Reporter: Dieter Wimberger > Attachments: AjaxConfigManagerAction.java.diff, FELIX-587-fmeschbe.patch > > > A Configuration instance for a ManagedServiceFactory will cause an Exception in org.apache.felix.webconsole.internal.compendium.AjaxConfigManagerAction.configForm(), because the implementation will try to obtain an ObjectClassDefinition for the Configuration pid, rather than for the corresponding factoryPid. > Fix: > The method should verify if a factoryPid is assigned to the Configuration instance and handle this case properly. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.