[MYFACES-3822] General cleanup and remove unused web config params

• Type: Task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.2.0, 2.3-next-M2
• Component/s: JSR-344
• Assignee: Leonardo Uribe
• Reporter: Leonardo Uribe
• Created: 08/Nov/13 00:07
• Updated: 28/Mar/20 18:56
• Resolved: 02/Jan/14 21:19

Description

This issue includes:

• Remove unused classes
• Refactor and remove duplicate code
• Move utility code from impl to shared

This issue also includes remove unused params like:

org.apache.myfaces.PRETTY_HTML
Reason : never worked well, modern browsers has support for debugging, makes rendering slow.

org.apache.myfaces.ALLOW_JAVASCRIPT
Reason: JSF 2 and upper versions require javascript. Now it is no longer optional, it is mandatory for JSF applications.

org.apache.myfaces.RENDER_FORM_SUBMIT_SCRIPT_INLINE
Reason: JSF 2 and upper versions require javascript. Now it is no longer optional, it is mandatory for JSF applications. Inline rendering is no longer required.

org.apache.myfaces.VIEWSTATE_JAVASCRIPT
Reason: Never worked.

org.apache.myfaces.HANDLE_STATE_CACHING_MECHANICS
Reason: Fix for MyFaces 1.2 state saving style in 2.0/2.1 . No need to support it anymore

org.apache.myfaces.SAVE_STATE_WITH_VISIT_TREE_ON_PSS
Reason: JSF 2.2 specify state saving should use visit tree call. No need to make it optional.

org.apache.myfaces.ERROR_HANDLER
Reason: deprecated by JSF 2.0 ExceptionHandler

Issue Links

• links to: GitHub Pull Request #79
• links to: GitHub Pull Request #81