[jira] Commented: (COCOON-1898) [PATCH] XPatch support for maven-cocoon-deployer-plugin

2008-02-19 Thread Olivier Billard (JIRA)

[ 
https://issues.apache.org/jira/browse/COCOON-1898?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12570496#action_12570496
 ] 

Olivier Billard commented on COCOON-1898:
-

Hi there,

This issue is quite old, and I also have the need of patching cocoon.xconf file 
in a mavenized Cocoon 2.1.x application.
I have several jar artefacts corresponding to different possible configurations 
(a base core, and customers-oriented/specific functionalities). 
Is there any chance this would be managed ?

Thanks !

 [PATCH] XPatch support for maven-cocoon-deployer-plugin
 ---

 Key: COCOON-1898
 URL: https://issues.apache.org/jira/browse/COCOON-1898
 Project: Cocoon
  Issue Type: Improvement
  Components: - Build System: Maven
Affects Versions: 2.2-dev (Current SVN)
Reporter: Lars Trieloff
 Attachments: maven-cocoon-deployer-plugin-with-xpatch-support.patch


 The cocoon-deployer-plugin has currently no support for XPatch, which makes 
 it difficult to modify the web.xml when developing cocoon blocks. For example 
 the cocoon-xmldb-impl block should add, when deployed, a servlet for xindice 
 and a servlet mapping for the xindice servlet. This was possible in 2.1 using 
 the XConfToolTask, but is no longer possible with the current state of the 
 deployer-plugin.
 My patch adds support for patching the web.xml file using *.xweb files in the 
 /conf directory of a block by filtering the block's jar file during 
 deployment for conf/*.xweb files, caching the patch document temporarily and 
 applying them (using code from the orgiginal XConfToolTask in 2.1) to the 
 web.xml. The patch has currently no support for other files than conf/*.xweb 
 files and does not support any property expansion.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



PipelineEvents eat children

2008-02-19 Thread solprovider
Classes extending SimpleParentProcessingNode decide which children to
process and call invoke() on those children.
Classes extending PipelineEventComponentProcessingNode assume all
children are configuration for the parent node.  Any children not
understood by the component are ignored.

Compare these examples:
map:select
   map:when test=first
  map:transform
 map:parameter name=choice value=first/
  /map:transform
   /map:when
   map:otherwise
  map:transform
 map:parameter name=choice value=other/
  /map:transform
   /map:otherwise
/map:select

And:
map:transform
   map:select
  map:when test=first
 map:parameter name=choice value=first/
  /map:when
  map:otherwise
 map:parameter name=choice value=other/
  /map:otherwise
   /map:select
/map:transform

The first example works because no ParentProcessingNodes are children
of a PipelineEvent.
The second transform will have the parameter choice default to the
empty string because the map:select element was ignored.

Differentiating the elements may make sense to Cocoon devs, but is not
obvious to Cocoon users.  The second example is obviously better code:
shorter with decision-making closer to the effect. The differentiation
only exists due to Cocoon storing the PipelineEvents for the second
phase of processing.

Workarounds are possible after a user learns about the limitations of
certain elements -- difficult since these limitations are not
mentioned on the Cocoon website.  The workarounds can become very
messy.  Imagine three Selectors each need to set parameters for one
transform.  Much code would be duplicated or an extra pipeline added
for each Selector.  Both methods reduce maintainability, and the
latter adds processing.

The obvious solution is for the PipelineEvent nodes to test each child
with isInvocable() [or isComponent() or isProcessingNode()?] and call
child.invoke() where appropriate.  This is feasible as only three
PipelineEvent classes exist: GenerateNode, TransformNode, and
SerializeNode.

Another (better?) solution would test child nodes before passing
control to the PipelineEvent nodes.

This affects Cocoon-2.1 through Cocoon-2.1.11.  Has this been fixed in
Cocoon-2.2?

Thoughts?

solprovider


[jira] Assigned: (COCOON-2020) CAPTCHA input element should have autocomplete=off

2008-02-19 Thread Antonio Gallardo (JIRA)

 [ 
https://issues.apache.org/jira/browse/COCOON-2020?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Antonio Gallardo reassigned COCOON-2020:


Assignee: Antonio Gallardo

 CAPTCHA input element  should have autocomplete=off
 ---

 Key: COCOON-2020
 URL: https://issues.apache.org/jira/browse/COCOON-2020
 Project: Cocoon
  Issue Type: Bug
  Components: Blocks: Forms
Affects Versions: 2.1.6, 2.1.7, 2.1.8, 2.1.9, 2.1.10, 2.1.11, 2.2-dev 
 (Current SVN)
Reporter: Mark Lundquist
Assignee: Antonio Gallardo
Priority: Trivial

 The bug summary says it all, and this isn't even enough to bother making a 
 patch, all it takes is the following lines added to forms-field-styling.xsl 
 (the diff is from my 2.1.8 vendor branch, but this fix will work in all 
 Cocoon versions):
 forms-field-styling.xsl(working copy)
 @@ -55,6 +55,9 @@
/xsl:if
!--  @id:input is what labels point to --
input name=[EMAIL PROTECTED] id=[EMAIL PROTECTED]:input 
 value={fi:value} title={fi:hint} type=text
 +xsl:if test=fi:captcha-image
 +  xsl:attribute name=autocomplete select='off'/
 +/xsl:if
  xsl:apply-templates select=. mode=styling/
/input
xsl:apply-templates select=. mode=common/

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Closed: (COCOON-2020) CAPTCHA input element should have autocomplete=off

2008-02-19 Thread Antonio Gallardo (JIRA)

 [ 
https://issues.apache.org/jira/browse/COCOON-2020?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Antonio Gallardo closed COCOON-2020.


   Resolution: Fixed
Fix Version/s: 2.2-dev (Current SVN)
   2.1.12-dev (Current SVN)

Thanks for the patch. :)

 CAPTCHA input element  should have autocomplete=off
 ---

 Key: COCOON-2020
 URL: https://issues.apache.org/jira/browse/COCOON-2020
 Project: Cocoon
  Issue Type: Bug
  Components: Blocks: Forms
Affects Versions: 2.1.6, 2.1.7, 2.1.8, 2.1.9, 2.1.10, 2.1.11, 2.2-dev 
 (Current SVN)
Reporter: Mark Lundquist
Assignee: Antonio Gallardo
Priority: Trivial
 Fix For: 2.1.12-dev (Current SVN), 2.2-dev (Current SVN)


 The bug summary says it all, and this isn't even enough to bother making a 
 patch, all it takes is the following lines added to forms-field-styling.xsl 
 (the diff is from my 2.1.8 vendor branch, but this fix will work in all 
 Cocoon versions):
 forms-field-styling.xsl(working copy)
 @@ -55,6 +55,9 @@
/xsl:if
!--  @id:input is what labels point to --
input name=[EMAIL PROTECTED] id=[EMAIL PROTECTED]:input 
 value={fi:value} title={fi:hint} type=text
 +xsl:if test=fi:captcha-image
 +  xsl:attribute name=autocomplete select='off'/
 +/xsl:if
  xsl:apply-templates select=. mode=styling/
/input
xsl:apply-templates select=. mode=common/

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.