You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@myfaces.apache.org by "Gabrielle Crawford (JIRA)" <de...@myfaces.apache.org> on 2008/10/15 00:23:44 UTC
[jira] Created: (TRINIDAD-1261) remove UIXCookie as the values are
no longer used
remove UIXCookie as the values are no longer used
-------------------------------------------------
Key: TRINIDAD-1261
URL: https://issues.apache.org/jira/browse/TRINIDAD-1261
Project: MyFaces Trinidad
Issue Type: Bug
Reporter: Gabrielle Crawford
UIXCookie seems to be dead code that should be removed.
UIXCookie is supposed to supports the following state:
* The accessibility mode chosen by the user
* The TimeZone of the user
As far as I can tell, the info on the cookie for the tz and accessibility mode are never used.
TimeZone
* Server: UIXCookie.setTimeZone, the only way to assign
the UIXCookie tz, is never called on the server.
* Client: There is a script written to the client in
HeadRenderer, which on the client writes tz info
into the uixcookie. However this value is never used
on either the client or server. It used to be used on
the server in RequestContextImpl.getTimeZone, but that
code has been commented out long ago.
* Conclusion: We're needlessly writing out a cookie script
when the value set on the client is never used.
Accessibility Mode
* Server: UIXCookie.setAccessibilityMode, the only way to
assign the UIXCookie accessibility mode, is never called
* Client: I see no references on the client to the
accessibility mode cookie value.
* Conclusion: I don't see how the mode could be set on
the cookie. Andy Schwartz, who has worked extensively on
accessibility issues, thinks this is old code that is no
longer in use.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
Re: TRINIDAD-1261 remove UIXCookie as the values are no longer used
Posted by Matthias Wessendorf <mw...@gmail.com>.
+1 on that clean up.
Sent from my iPod.
Am 15.10.2008 um 00:31 schrieb Gabrielle Crawford <gabrielle.crawford@oracle.com
>:
> Hi,
>
> As far as I can tell, the UIXCookie tz and accessibility mode values
> are either never set, or if set, never used. I believe both were
> used in the past but have become obsolete. I'm therefore planning to
> remove UIXCookie.
>
> Let me know if there are objections.
>
> Thanks,
>
> Gabrielle
>
> Gabrielle Crawford (JIRA) wrote:
>> remove UIXCookie as the values are no longer used
>> -------------------------------------------------
>>
>> Key: TRINIDAD-1261
>> URL: https://issues.apache.org/jira/browse/TRINIDAD-1261
>> Project: MyFaces Trinidad
>> Issue Type: Bug
>> Reporter: Gabrielle Crawford
>>
>>
>> UIXCookie seems to be dead code that should be removed.
>>
>> UIXCookie is supposed to supports the following state:
>>
>> * The accessibility mode chosen by the user
>> * The TimeZone of the user
>>
>>
>> As far as I can tell, the info on the cookie for the tz and
>> accessibility mode are never used.
>>
>> TimeZone
>>
>> * Server: UIXCookie.setTimeZone, the only way to assign
>> the UIXCookie tz, is never called on the server.
>> * Client: There is a script written to the client in
>> HeadRenderer, which on the client writes tz info
>> into the uixcookie. However this value is never used
>> on either the client or server. It used to be used on
>> the server in RequestContextImpl.getTimeZone, but that
>> code has been commented out long ago.
>> * Conclusion: We're needlessly writing out a cookie script
>> when the value set on the client is never used.
>>
>>
>> Accessibility Mode
>>
>> * Server: UIXCookie.setAccessibilityMode, the only way to
>> assign the UIXCookie accessibility mode, is never called
>> * Client: I see no references on the client to the
>> accessibility mode cookie value.
>> * Conclusion: I don't see how the mode could be set on
>> the cookie. Andy Schwartz, who has worked extensively
>> on accessibility issues, thinks this is old code that is no
>> longer in use.
>>
>>
>>
TRINIDAD-1261 remove UIXCookie as the values are no longer used
Posted by Gabrielle Crawford <ga...@oracle.com>.
Hi,
As far as I can tell, the UIXCookie tz and accessibility mode values are
either never set, or if set, never used. I believe both were used in the
past but have become obsolete. I'm therefore planning to remove UIXCookie.
Let me know if there are objections.
Thanks,
Gabrielle
Gabrielle Crawford (JIRA) wrote:
> remove UIXCookie as the values are no longer used
> -------------------------------------------------
>
> Key: TRINIDAD-1261
> URL: https://issues.apache.org/jira/browse/TRINIDAD-1261
> Project: MyFaces Trinidad
> Issue Type: Bug
> Reporter: Gabrielle Crawford
>
>
> UIXCookie seems to be dead code that should be removed.
>
> UIXCookie is supposed to supports the following state:
>
> * The accessibility mode chosen by the user
> * The TimeZone of the user
>
>
> As far as I can tell, the info on the cookie for the tz and accessibility mode are never used.
>
> TimeZone
>
> * Server: UIXCookie.setTimeZone, the only way to assign
> the UIXCookie tz, is never called on the server.
> * Client: There is a script written to the client in
> HeadRenderer, which on the client writes tz info
> into the uixcookie. However this value is never used
> on either the client or server. It used to be used on
> the server in RequestContextImpl.getTimeZone, but that
> code has been commented out long ago.
> * Conclusion: We're needlessly writing out a cookie script
> when the value set on the client is never used.
>
>
> Accessibility Mode
>
> * Server: UIXCookie.setAccessibilityMode, the only way to
> assign the UIXCookie accessibility mode, is never called
> * Client: I see no references on the client to the
> accessibility mode cookie value.
> * Conclusion: I don't see how the mode could be set on
> the cookie. Andy Schwartz, who has worked extensively on
> accessibility issues, thinks this is old code that is no
> longer in use.
>
>
>
[jira] Resolved: (TRINIDAD-1261) remove UIXCookie as the values are
no longer used
Posted by "Gabrielle Crawford (JIRA)" <de...@myfaces.apache.org>.
[ https://issues.apache.org/jira/browse/TRINIDAD-1261?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Gabrielle Crawford resolved TRINIDAD-1261.
------------------------------------------
Resolution: Fixed
Fix Version/s: 1.2.10-core
> remove UIXCookie as the values are no longer used
> -------------------------------------------------
>
> Key: TRINIDAD-1261
> URL: https://issues.apache.org/jira/browse/TRINIDAD-1261
> Project: MyFaces Trinidad
> Issue Type: Bug
> Reporter: Gabrielle Crawford
> Assignee: Gabrielle Crawford
> Fix For: 1.2.10-core
>
>
> UIXCookie seems to be dead code that should be removed.
> UIXCookie is supposed to supports the following state:
> * The accessibility mode chosen by the user
> * The TimeZone of the user
> As far as I can tell, the info on the cookie for the tz and accessibility mode are never used.
> TimeZone
> * Server: UIXCookie.setTimeZone, the only way to assign
> the UIXCookie tz, is never called on the server.
> * Client: There is a script written to the client in
> HeadRenderer, which on the client writes tz info
> into the uixcookie. However this value is never used
> on either the client or server. It used to be used on
> the server in RequestContextImpl.getTimeZone, but that
> code has been commented out long ago.
> * Conclusion: We're needlessly writing out a cookie script
> when the value set on the client is never used.
> Accessibility Mode
> * Server: UIXCookie.setAccessibilityMode, the only way to
> assign the UIXCookie accessibility mode, is never called
> * Client: I see no references on the client to the
> accessibility mode cookie value.
> * Conclusion: I don't see how the mode could be set on
> the cookie. Andy Schwartz, who has worked extensively on
> accessibility issues, thinks this is old code that is no
> longer in use.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (TRINIDAD-1261) remove UIXCookie as the values
are no longer used
Posted by "Gabrielle Crawford (JIRA)" <de...@myfaces.apache.org>.
[ https://issues.apache.org/jira/browse/TRINIDAD-1261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12641958#action_12641958 ]
Gabrielle Crawford commented on TRINIDAD-1261:
----------------------------------------------
fixed on 1.2 branch in rev 707157
> remove UIXCookie as the values are no longer used
> -------------------------------------------------
>
> Key: TRINIDAD-1261
> URL: https://issues.apache.org/jira/browse/TRINIDAD-1261
> Project: MyFaces Trinidad
> Issue Type: Bug
> Reporter: Gabrielle Crawford
> Assignee: Gabrielle Crawford
> Fix For: 1.2.10-core
>
>
> UIXCookie seems to be dead code that should be removed.
> UIXCookie is supposed to supports the following state:
> * The accessibility mode chosen by the user
> * The TimeZone of the user
> As far as I can tell, the info on the cookie for the tz and accessibility mode are never used.
> TimeZone
> * Server: UIXCookie.setTimeZone, the only way to assign
> the UIXCookie tz, is never called on the server.
> * Client: There is a script written to the client in
> HeadRenderer, which on the client writes tz info
> into the uixcookie. However this value is never used
> on either the client or server. It used to be used on
> the server in RequestContextImpl.getTimeZone, but that
> code has been commented out long ago.
> * Conclusion: We're needlessly writing out a cookie script
> when the value set on the client is never used.
> Accessibility Mode
> * Server: UIXCookie.setAccessibilityMode, the only way to
> assign the UIXCookie accessibility mode, is never called
> * Client: I see no references on the client to the
> accessibility mode cookie value.
> * Conclusion: I don't see how the mode could be set on
> the cookie. Andy Schwartz, who has worked extensively on
> accessibility issues, thinks this is old code that is no
> longer in use.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.