search

hhuijser / evp-portlet-upgrade

0 stars 0 forks source link

Test0309 #84

Closed hhuijser closed 3 years ago

lugbot commented 3 years ago

Found 28 formatting issues

MapIterationCheck, click here for documentation

Issues - Use Map.entrySet() to retrieve values: [/actions.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portal/api/jsonws/actions.jsp#L75) - Use Map.entrySet() to retrieve values: [/asset_selection.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portlet/asset_publisher/asset_selection.jsp#L146) - Use Map.entrySet() to retrieve values: [/view.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portlet/control_panel_home/view.jsp#L25) - Use Map.entrySet() to retrieve values: [/view.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portlet/control_panel_menu/view.jsp#L45) - Use Map.entrySet() to retrieve values: [/look_and_feel_themes.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portlet/layouts_admin/look_and_feel_themes.jsp#L143) - Use Map.entrySet() to retrieve values: [/view.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portlet/social_activity/view.jsp#L46)

StringCastCheck

Issues - No need to call '.toString()' on type String: [/render_portlet.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portal/render_portlet.jsp#L591) - No need to call '.toString()' on type String: [/facebook.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portlet/login/navigation/facebook.jsp#L45)

UnnecessaryTypeCastCheck

Issues - No need to apply Type Cast to method 'groups.get': [/edit_role_permissions_resource.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portlet/roles_admin/edit_role_permissions_resource.jsp#L131) - No need to apply Type Cast to method 'portlets.get': [/open_search.jspf](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portlet/search/open_search.jspf#L69) - No need to apply Type Cast to method 'portlets.get': [/open_search.jspf](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portlet/search/open_search.jspf#L99) - No need to apply Type Cast to method 'results.get': [/page_iterator.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portlet/wiki/page_iterator.jsp#L379) - No need to apply Type Cast to method 'messages.get': [/page.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/taglib/ui/discussion/page.jsp#L231)

ValidatorEqualsCheck, click here for documentation

Issues - Use Objects.equals(Object, Object) instead of Validator.equals(Object, Object): [/ckeditor_bbcode.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/js/editor/ckeditor_bbcode.jsp#L24) - Use Objects.equals(Object, Object) instead of Validator.equals(Object, Object): [/ckeditor_creole.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/js/editor/ckeditor_creole.jsp#L24) - Use Objects.equals(Object, Object) instead of Validator.equals(Object, Object): [/edit_query_rule.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portlet/asset_publisher/edit_query_rule.jsp#L37) - Use Objects.equals(Object, Object) instead of Validator.equals(Object, Object): [/view_entry_content.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portlet/blogs/view_entry_content.jsp#L170) - Use Objects.equals(Object, Object) instead of Validator.equals(Object, Object): [/edit_template.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portlet/dynamic_data_mapping/edit_template.jsp#L63) - Use Objects.equals(Object, Object) instead of Validator.equals(Object, Object): [/edit_template_display.jspf](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portlet/dynamic_data_mapping/edit_template_display.jspf#L633) - Use Objects.equals(Object, Object) instead of Validator.equals(Object, Object): [/publish_process_message.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portlet/layouts_admin/publish_process_message.jsp#L57) - Use Objects.equals(Object, Object) instead of Validator.equals(Object, Object): [/publish_process_message.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portlet/layouts_admin/publish_process_message.jsp#L89) - Use Objects.equals(Object, Object) instead of Validator.equals(Object, Object): [/publish_process_message.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portlet/layouts_admin/publish_process_message.jsp#L92) - Use Objects.equals(Object, Object) instead of Validator.equals(Object, Object): [/display_settings.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portlet/portal_settings/display_settings.jsp#L139) - Use Objects.equals(Object, Object) instead of Validator.equals(Object, Object): [/display_settings.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portlet/portal_settings/display_settings.jsp#L166) - Use Objects.equals(Object, Object) instead of Validator.equals(Object, Object): [/display_settings.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portlet/portal_settings/display_settings.jsp#L199) - Use Objects.equals(Object, Object) instead of Validator.equals(Object, Object): [/display_settings.jsp](https://github.com/hhuijser/evp-portlet-upgrade/blob/test0309/liferay-portal-6.2-ee-sp20/tomcat-7.0.62/webapps/ROOT/html/portlet/portal_settings/display_settings.jsp#L203)