[ https://issues.apache.org/jira/browse/HBASE-13056?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14325473#comment-14325473 ]
Vikas Vishwakarma commented on HBASE-13056: ------------------------------------------- Also [~stack] [~apurtell] formatting is a bit iffy, if i fix it entire file will show as diff. Let me know if you want me to take that up in this patch. I will fix it for the whole file. > Refactor table.jsp code to remove repeated code and make it easier to add new > checks > ------------------------------------------------------------------------------------ > > Key: HBASE-13056 > URL: https://issues.apache.org/jira/browse/HBASE-13056 > Project: HBase > Issue Type: Improvement > Affects Versions: 2.0.0 > Reporter: Vikas Vishwakarma > Assignee: Vikas Vishwakarma > Fix For: 2.0.0 > > Attachments: HBASE-13056.patch > > > While trying to fix HBASE-13001, I realized that there is lot of html code > repetition in table.jsp which is making addition of new checks slightly > difficult in the sense I will have to: > 1. Add the check at multiple places in the code > Or > 2. Repeat the html code again for the new check > So I am proposing to re-factor table.jsp code such that the common html > header/body is loaded without any condition check and then we generate the > condition specific html code > snapshot.jsp follows the same format as explained below: > {noformat} > Current implementation: > ==================== > if( x ) { > title_x > common_html_header > common_html_body > x_specific_html_body > } else { > title_y > common_html_header > common_html_body > y_specific_html_body > } > New Implementation: > ================== > if( x ) { > title_x > } else { > title_y > } > common_html_header > common_html_body > if( x ) { > x_specific_html_body > } else { > y_specific_html_body > } > {noformat} -- This message was sent by Atlassian JIRA (v6.3.4#6332)