Re: BRANCH_2_1_X-dojo1_1 - dojoId is not defined

2010-07-14 Thread Carlos Chávez
Jeroen Reijn Escribio :-)
 Hi,

 there was some effort of updating to dojo 1.1, but the work in the
 branch as far as I know is not finished.

 Jeroen

Hi Jeroen.
thanks for the feedback.

I'm trying to help in the testing, I know Jeremy work sometime ago
on this branch.

So, in anycase I'm attaching another patch, this patch is to make work
again the suggestion list of type javascript:

fd:suggestion-list type=javascript

--
Cheers.
Carlos Chávez.


 On Mon, Jul 5, 2010 at 6:56 AM, Carlos Chávez ccha...@agssa.net wrote:
 Hi guys.

 I was testing the branch
 http://svn.apache.org/repos/asf/cocoon/branches/BRANCH_2_1_X-dojo1_1. I
 enter in the sample:

 http://localhost:/samples/blocks/forms/do-calculatedfields.flow

 I tried to add a row to the repeater and it shows the error:

 dojoId is not defined.

 I attached to the email the fix for this error.

 --
 Cheers.
 Carlos Chávez.


### Eclipse Workspace Patch 1.0
#P BRANCH_2_1_X-dojo1_1
Index: 
src/blocks/forms/resources/org/apache/cocoon/forms/resources/forms-advanced-field-styling.xsl
===
--- 
src/blocks/forms/resources/org/apache/cocoon/forms/resources/forms-advanced-field-styling.xsl
   (revision 960158)
+++ 
src/blocks/forms/resources/org/apache/cocoon/forms/resources/forms-advanced-field-styling.xsl
   (working copy)
@@ -47,13 +47,14 @@
   xsl:param name=dojo-resourcesxsl:value-of 
select=$context-path//_cocoon/resources/dojotoolkit/xsl:param !-- Url 
prefix for dojo resources --
   xsl:param name=forms-resourcesxsl:value-of 
select=$context-path//_cocoon/resources/forms/xsl:param  !-- Url 
prefix for forms block resources --
   xsl:param name=forms-systemxsl:value-of 
select=$context-path//_cocoon/system/forms/xsl:param!-- Url 
prefix for forms block system pipelines --
-  
+  xsl:param 
name=forms-suggest_cocoon/system/forms/suggestionlist/xsl:param
   !-- Special case for the suggest widget --
+
   !-- set the default dojo css theme, other choices are currently 'nihilo' 
and 'soria' --
   xsl:param name=dojo-theme-defaulttundra/xsl:param
-  
+
   !-- A convenient way for samples to have a url that switches the theme. If 
your app does not want this, don't send this param from your sitemap --
   xsl:param name=dojo-theme-param/xsl:param
-  
+
   !-- the determine dojo css theme to use --
   xsl:variable name=dojo-theme
 xsl:choose
@@ -61,15 +62,15 @@
 xsl:otherwisexsl:value-of 
select=$dojo-theme-param//xsl:otherwise
 /xsl:choose
   /xsl:variable
-  
+
   !-- option to turn on console debugging for dojo on the browser and loading 
of uncompressed resources, from a parameter in the sitemap --
   xsl:param name=dojo-debugfalse/xsl:param
   xsl:variable name=dojo-debug-jsxsl:if 
test=$dojo-debug='true'.uncompressed.js/xsl:if/xsl:variable!-- load the 
uncompressed version for debug mode --
   xsl:variable name=dojo-debug-cssxsl:if 
test=$dojo-debug='true'.commented.css/xsl:if/xsl:variable
-  
+
   !-- Configure the dojo locale from a parameter in the sitemap (required?). 
--
-  xsl:param name=dojo-locale/xsl:param 
-  
+  xsl:param name=dojo-locale/xsl:param
+
   !-- Create a variable with the normalized locale, dojo needs locale parts 
to be separated with a dash --
   xsl:variable name=dojoLocale
 xsl:choose
@@ -81,7 +82,7 @@
   /xsl:otherwise
 /xsl:choose
   /xsl:variable
-  
+
   !-- create a variable for the dojo configuration --
   xsl:variable name=djConfig
 isDebug: xsl:value-of select=$dojo-debug/,
@@ -94,11 +95,11 @@
 locale: 'xsl:value-of select=$dojoLocale/'xsl:if test=$dojo-use-cdn 
= 'true' and $dojo-cdn != '',
 dojoBlankHtmlUrl: 'xsl:value-of 
select=$dojo-resources//dojo/resources/blank.html',
 dojoIframeHistoryUrl: 'xsl:value-of 
select=$dojo-resources//dojo/resources/iframe_history.html', !-- not 
currently used by cforms itself --
-baseUrl: './', 
+baseUrl: './',
 xdWaitSeconds: 10/xsl:if
   /xsl:variable
 
-  xsl:template match=head mode=forms-field
+  xsl:template match=head mode=forms-field
 xsl:choose
   xsl:when test=$dojo-use-cdn = 'true' and $dojo-cdn != ''
 link rel=stylesheet type=text/css 
href={$dojo-cdn}/dijit/themes/dijit.css/!-- Google CDN does not keep 
*.commented.css --
@@ -146,11 +147,11 @@
   | Adds support for live data validation on the client, based on 
fi:datatype
   | (Browser validation may be turned off with 
fi:styling/@browserValidation=false)
   +--
-  
+
   !-- TODO: Document new options --
   !-- TODO: fi:styling/@regExp should not be a user tag, it should come 
from the validator (eventually?) --
-  
-  xsl:template match=fi:field[not(fi:styling/@type='inplace-area') and 
(@state='active' or @state='disabled')] | 
+
+  xsl:template match=fi:field[not(fi:styling/@type='inplace-area') and 
(@state='active' or @state='disabled')] |

Re: BRANCH_2_1_X-dojo1_1 - dojoId is not defined

2010-07-05 Thread Jeroen Reijn
Hi,

there was some effort of updating to dojo 1.1, but the work in the
branch as far as I know is not finished.

Jeroen

On Mon, Jul 5, 2010 at 6:56 AM, Carlos Chávez ccha...@agssa.net wrote:
 Hi guys.

 I was testing the branch
 http://svn.apache.org/repos/asf/cocoon/branches/BRANCH_2_1_X-dojo1_1. I
 enter in the sample:

 http://localhost:/samples/blocks/forms/do-calculatedfields.flow

 I tried to add a row to the repeater and it shows the error:

 dojoId is not defined.

 I attached to the email the fix for this error.

 --
 Cheers.
 Carlos Chávez.



BRANCH_2_1_X-dojo1_1 - dojoId is not defined

2010-07-04 Thread Carlos Chávez
Hi guys.

I was testing the branch
http://svn.apache.org/repos/asf/cocoon/branches/BRANCH_2_1_X-dojo1_1. I
enter in the sample:

http://localhost:/samples/blocks/forms/do-calculatedfields.flow

I tried to add a row to the repeater and it shows the error:

dojoId is not defined.

I attached to the email the fix for this error.

--
Cheers.
Carlos Chávez.
### Eclipse Workspace Patch 1.0
#P BRANCH_2_1_X-dojo1_1
Index: src/blocks/forms/resources/org/apache/cocoon/forms/resources/js/common.js
===
--- src/blocks/forms/resources/org/apache/cocoon/forms/resources/js/common.js   
(revision 960158)
+++ src/blocks/forms/resources/org/apache/cocoon/forms/resources/js/common.js   
(working copy)
@@ -71,7 +71,7 @@
 var formId = form.id;
 if (dojo  formId) {
 // Delegate to the SimpleForm or AjaxForm widget
-dijit.byId(dojoId).submit(name, params);
+dijit.byId(formId).submit(name, params);
 } else {
 // Regular submit. There is no *Form widget available
 
@@ -218,7 +218,7 @@
  */
 
 cocoon.forms.defaults = {
-
+
 statusMark: *,
 constraints: { // default constraints for numerical datatypes
 decimal:   {pattern:###0.###, type: decimal, locale:en}, 
// PlainDecimalConvertor NB. Dojo has no concept of a plain number, hence force 
it to show 'en' to get '.' decimal
@@ -226,7 +226,7 @@
 float: {pattern:###0.###, type: decimal, locale:en}, 
// PlainFloatConvertor
 integer:   {pattern: '#', places: 0, min: -2147483648, max: 
2147483647}, // PlainIntegerConvertor
 long:  {pattern: '#', places: 0, min: -9223372036854775808, 
max: 9223372036854775807},// PlainLongConvertor
-l10n-currency: {fractional:true},  // FormattingDecimalConverter - 
currency variant 
+l10n-currency: {fractional:true},  // FormattingDecimalConverter - 
currency variant
 l10n-integer:  {places: 0, min: -2147483648, max: 2147483647}, // 
FormattingIntegerConverter
 l10n-long: {places: 0, min:-9223372036854775808, max: 
9223372036854775807}, // FormattingLongConverter
 l10n-number:   {type: decimal}, // FormattingDecimalConverter, 
FormattingFloatConverter, FormattingDoubleConverter