You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@myfaces.apache.org by Dan Allen <da...@gmail.com> on 2006/06/09 22:26:57 UTC

bug in tomahawk HtmlTableRenderer (with patch)

There is a typo in the HtmlTableRenderer for tomahawk that applies the
rowStyle value where it should be applying the rowStyleClass value.

Here is the patch:

--- HtmlTableRenderer.java      2006-06-09 16:24:47.000000000 -0400
+++ HtmlTableRenderer.java.fixed        2006-06-09 16:25:15.000000000 -0400
@@ -443,7 +443,7 @@
         }
         else
         {
-            writer.writeAttribute(HTML.CLASS_ATTR, rowStyle, null);
+            writer.writeAttribute(HTML.CLASS_ATTR, rowStyleClass, null);
         }
         if(rowStyle != null)
         {


-- 
Daniel Allen
Registered Linux User #231597

Mojavelinux.com: Open Source Advocacy
http://www.mojavelinux.com

While I make a strong effort to keep up with my email on a daily basis,
life and work come first and, at times, keep me away from my mail for a
while. If you contact me and then don't hear back for more than a week,
it is very likely that I am excessively backlogged or the message was
caught in the filters.  Please don't hesitate to resend a message if
you feel that it did not reach my attention.

Re: bug in tomahawk HtmlTableRenderer (with patch)

Posted by Matthias Wessendorf <ma...@apache.org>.
thx for heads up.

but please use in future our jira ticket system, so bugs / typs don't get lost.

-Matthias

On 6/9/06, Dan Allen <da...@gmail.com> wrote:
> There is a typo in the HtmlTableRenderer for tomahawk that applies the
> rowStyle value where it should be applying the rowStyleClass value.
>
> Here is the patch:
>
> --- HtmlTableRenderer.java      2006-06-09 16:24:47.000000000 -0400
> +++ HtmlTableRenderer.java.fixed        2006-06-09 16:25:15.000000000 -0400
> @@ -443,7 +443,7 @@
>          }
>          else
>          {
> -            writer.writeAttribute(HTML.CLASS_ATTR, rowStyle, null);
> +            writer.writeAttribute(HTML.CLASS_ATTR, rowStyleClass, null);
>          }
>          if(rowStyle != null)
>          {
>
>
> --
> Daniel Allen
> Registered Linux User #231597
>
> Mojavelinux.com: Open Source Advocacy
> http://www.mojavelinux.com
>
> While I make a strong effort to keep up with my email on a daily basis,
> life and work come first and, at times, keep me away from my mail for a
> while. If you contact me and then don't hear back for more than a week,
> it is very likely that I am excessively backlogged or the message was
> caught in the filters.  Please don't hesitate to resend a message if
> you feel that it did not reach my attention.
>


-- 
Matthias Wessendorf
Aechterhoek 18
48282 Emsdetten
blog: http://jroller.com/page/mwessendorf
mail: mwessendorf-at-gmail-dot-com