Return-Path: Delivered-To: apmail-click-dev-archive@www.apache.org Received: (qmail 95270 invoked from network); 22 Jun 2010 17:55:21 -0000 Received: from unknown (HELO mail.apache.org) (140.211.11.3) by 140.211.11.9 with SMTP; 22 Jun 2010 17:55:21 -0000 Received: (qmail 93682 invoked by uid 500); 22 Jun 2010 17:55:21 -0000 Delivered-To: apmail-click-dev-archive@click.apache.org Received: (qmail 93624 invoked by uid 500); 22 Jun 2010 17:55:21 -0000 Mailing-List: contact dev-help@click.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@click.apache.org Delivered-To: mailing list dev@click.apache.org Received: (qmail 93613 invoked by uid 99); 22 Jun 2010 17:55:21 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 22 Jun 2010 17:55:21 +0000 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.22] (HELO thor.apache.org) (140.211.11.22) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 22 Jun 2010 17:55:19 +0000 Received: from thor (localhost [127.0.0.1]) by thor.apache.org (8.13.8+Sun/8.13.8) with ESMTP id o5MHsvZF019438 for ; Tue, 22 Jun 2010 17:54:57 GMT Message-ID: <15953752.17161277229297125.JavaMail.jira@thor> Date: Tue, 22 Jun 2010 13:54:57 -0400 (EDT) From: "Md. Jahid Shohel (JIRA)" To: dev@click.apache.org Subject: [jira] Commented: (CLK-661) Add Java based configuration In-Reply-To: <30163817.21741271469864852.JavaMail.jira@thor> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 X-Virus-Checked: Checked by ClamAV on apache.org [ https://issues.apache.org/jira/browse/CLK-661?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12881264#action_12881264 ] Md. Jahid Shohel commented on CLK-661: -------------------------------------- I think I could not make my self clear totally. We need the base class, and I agree with that. So that sub classes can override that. And about the matter I was talking. How you will mix up two config service to work together. Say as an example, xmlconfigservice and annotationconfigservice. Say a user defined his autobinding configuration using annotation, and rest of the configuration using xml configuration. On this situation, before xmlconfigservice loads up the pages, the system need to process annotation config service, and then letting xmlconfigservice to load pages. Sequence wise it will be something like - 1) "xmlconfigservice.loadAutobinding" and then "annotationconfigservice.loadAutobinding" (where annotation will check if autobinding is not set then initialize it) 2) "xmlconfigservice.loadPages" and then "annotationconfigservice.loadPages". On this case, we mixing two config service working side by side. Because there will be no point of loading autobinding configuration declared on annotation after xmlconfigservice loads up the pages. I am not sure if I was able to make the case clear. But this is the case I faced to integrating annotation. > Add Java based configuration > ---------------------------- > > Key: CLK-661 > URL: https://issues.apache.org/jira/browse/CLK-661 > Project: Click > Issue Type: Improvement > Reporter: Bob Schellink > Assignee: Finn Bock > Fix For: 2.3.0-M1 > > Attachments: defaultconfigservice.patch > > > click.xml has grown over the years especially since we introduced the service based architecture. Some of the problems with xml based configurations is: > - no compile time checking > - no JavaDoc help in IDE > I propose we add a new DefaultConfigService class that is Java based which XmlConfigService extends from. > Its advantages is the polar opposite of xml disadvantage: > - compile time checking > - JavaDoc help in IDE > - More powerful configuration options, e.g. its possible to configure FileUpload in ways not exposed by xml config. Also possible to create more powerful algorithms e.g. which templates to include/exclude. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.