From 420e6f59649766bc72c9c9422dd08def637196ee Mon Sep 17 00:00:00 2001 From: Lars Hvam Date: Sat, 13 Jun 2020 09:39:46 +0200 Subject: [PATCH] replace HTML with interface (#3490) * replace HTML with interface part of #3460 * pretty print --- src/ui/core/zcl_abapgit_html.clas.abap | 61 +++++---- .../zcl_abapgit_html.clas.testclasses.abap | 4 +- src/ui/db/zcl_abapgit_gui_page_db.clas.abap | 46 +++---- .../db/zcl_abapgit_gui_page_db_dis.clas.abap | 18 +-- .../db/zcl_abapgit_gui_page_db_edit.clas.abap | 26 ++-- src/ui/zcl_abapgit_gui_page.clas.abap | 126 +++++++++--------- src/ui/zcl_abapgit_gui_page_bkg.clas.abap | 4 +- src/ui/zcl_abapgit_gui_page_bkg_run.clas.abap | 8 +- .../zcl_abapgit_gui_page_boverview.clas.abap | 8 +- .../zcl_abapgit_gui_page_code_insp.clas.abap | 22 +-- .../zcl_abapgit_gui_page_codi_base.clas.abap | 26 ++-- src/ui/zcl_abapgit_gui_page_commit.clas.abap | 14 +- .../zcl_abapgit_gui_page_debuginfo.clas.abap | 14 +- src/ui/zcl_abapgit_gui_page_diff.clas.abap | 12 +- src/ui/zcl_abapgit_gui_page_main.clas.abap | 60 ++++----- src/ui/zcl_abapgit_gui_page_merge.clas.abap | 96 ++++++------- .../zcl_abapgit_gui_page_merge_res.clas.abap | 8 +- src/ui/zcl_abapgit_gui_page_patch.clas.abap | 2 +- .../zcl_abapgit_gui_page_repo_over.clas.abap | 86 ++++++------ .../zcl_abapgit_gui_page_repo_sett.clas.abap | 76 +++++------ .../zcl_abapgit_gui_page_settings.clas.abap | 48 +++---- src/ui/zcl_abapgit_gui_page_stage.clas.abap | 20 +-- src/ui/zcl_abapgit_gui_page_syntax.clas.abap | 14 +- src/ui/zcl_abapgit_gui_page_tag.clas.abap | 10 +- src/ui/zcl_abapgit_hotkeys.clas.abap | 111 +++++++-------- 25 files changed, 458 insertions(+), 462 deletions(-) diff --git a/src/ui/core/zcl_abapgit_html.clas.abap b/src/ui/core/zcl_abapgit_html.clas.abap index ace487c1f..4b2f38d2c 100644 --- a/src/ui/core/zcl_abapgit_html.clas.abap +++ b/src/ui/core/zcl_abapgit_html.clas.abap @@ -1,33 +1,37 @@ CLASS zcl_abapgit_html DEFINITION PUBLIC - CREATE PUBLIC. + CREATE PUBLIC . PUBLIC SECTION. - INTERFACES zif_abapgit_html. - ALIASES: - add FOR zif_abapgit_html~add, - render FOR zif_abapgit_html~render, - is_empty FOR zif_abapgit_html~is_empty, - add_a FOR zif_abapgit_html~add_a, - add_checkbox FOR zif_abapgit_html~add_checkbox, - a FOR zif_abapgit_html~a, - icon FOR zif_abapgit_html~icon. + INTERFACES zif_abapgit_html . + + ALIASES a + FOR zif_abapgit_html~a . + ALIASES add + FOR zif_abapgit_html~add . + ALIASES add_a + FOR zif_abapgit_html~add_a . + ALIASES add_checkbox + FOR zif_abapgit_html~add_checkbox . + ALIASES icon + FOR zif_abapgit_html~icon . + ALIASES is_empty + FOR zif_abapgit_html~is_empty . + ALIASES render + FOR zif_abapgit_html~render . CONSTANTS c_indent_size TYPE i VALUE 2 ##NO_TEXT. - CLASS-METHODS class_constructor. + CLASS-METHODS class_constructor . METHODS add_icon IMPORTING !iv_name TYPE string !iv_hint TYPE string OPTIONAL !iv_class TYPE string OPTIONAL - !iv_onclick TYPE string OPTIONAL. + !iv_onclick TYPE string OPTIONAL . PROTECTED SECTION. PRIVATE SECTION. - CLASS-DATA: go_single_tags_re TYPE REF TO cl_abap_regex. - - DATA: mt_buffer TYPE string_table. TYPES: BEGIN OF ty_indent_context, @@ -36,8 +40,8 @@ CLASS zcl_abapgit_html DEFINITION within_js TYPE abap_bool, indent TYPE i, indent_str TYPE string, - END OF ty_indent_context, - + END OF ty_indent_context . + TYPES: BEGIN OF ty_study_result, style_open TYPE abap_bool, style_close TYPE abap_bool, @@ -48,26 +52,27 @@ CLASS zcl_abapgit_html DEFINITION openings TYPE i, closings TYPE i, singles TYPE i, - END OF ty_study_result. + END OF ty_study_result . + + CLASS-DATA go_single_tags_re TYPE REF TO cl_abap_regex . + DATA mt_buffer TYPE string_table . METHODS indent_line CHANGING - cs_context TYPE ty_indent_context - cv_line TYPE string. - + !cs_context TYPE ty_indent_context + !cv_line TYPE string . METHODS study_line IMPORTING - iv_line TYPE string - is_context TYPE ty_indent_context + !iv_line TYPE string + !is_context TYPE ty_indent_context RETURNING - VALUE(rs_result) TYPE ty_study_result. + VALUE(rs_result) TYPE ty_study_result . METHODS checkbox IMPORTING - iv_id TYPE string - iv_checked TYPE abap_bool OPTIONAL + !iv_id TYPE string + !iv_checked TYPE abap_bool OPTIONAL RETURNING - VALUE(rv_html) TYPE string. - + VALUE(rv_html) TYPE string . ENDCLASS. diff --git a/src/ui/core/zcl_abapgit_html.clas.testclasses.abap b/src/ui/core/zcl_abapgit_html.clas.testclasses.abap index 05d389346..648bfd0a2 100644 --- a/src/ui/core/zcl_abapgit_html.clas.testclasses.abap +++ b/src/ui/core/zcl_abapgit_html.clas.testclasses.abap @@ -2,7 +2,7 @@ CLASS ltcl_html DEFINITION FOR TESTING DURATION SHORT RISK LEVEL HARMLESS. PRIVATE SECTION. - DATA: mo_html TYPE REF TO zcl_abapgit_html. + DATA: mo_html TYPE REF TO zif_abapgit_html. METHODS: indent1 FOR TESTING RAISING zcx_abapgit_exception, @@ -20,7 +20,7 @@ ENDCLASS. CLASS ltcl_html IMPLEMENTATION. METHOD setup. - CREATE OBJECT mo_html. + CREATE OBJECT mo_html TYPE zcl_abapgit_html. ENDMETHOD. METHOD indent1. diff --git a/src/ui/db/zcl_abapgit_gui_page_db.clas.abap b/src/ui/db/zcl_abapgit_gui_page_db.clas.abap index 4a1f325ca..59a671e65 100644 --- a/src/ui/db/zcl_abapgit_gui_page_db.clas.abap +++ b/src/ui/db/zcl_abapgit_gui_page_db.clas.abap @@ -140,21 +140,21 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_DB IMPLEMENTATION. lt_data = zcl_abapgit_persistence_db=>get_instance( )->list( ). - CREATE OBJECT ro_html. + CREATE OBJECT ri_html TYPE zcl_abapgit_html. - ro_html->add( '
' ). - ro_html->add( '' ). + ri_html->add( '
' ). + ri_html->add( '
' ). " Header - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). " Lines LOOP AT lt_data ASSIGNING . @@ -173,19 +173,19 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_DB IMPLEMENTATION. lo_toolbar->add( iv_txt = 'Delete' iv_act = |{ c_action-delete }?{ lv_action }| ). - ro_html->add( || ). - ro_html->add( || ). - ro_html->add( || ). - ro_html->add( || ). - ro_html->add( '' ). - ro_html->add( '' ). + ri_html->add( || ). + ri_html->add( || ). + ri_html->add( || ). + ri_html->add( || ). + ri_html->add( '' ). + ri_html->add( '' ). ENDLOOP. - ro_html->add( '' ). - ro_html->add( '
TypeKeyData
TypeKeyData
{ -type }{ -value }{ explain_content( ) }' ). - ro_html->add( lo_toolbar->render( ) ). - ro_html->add( '
{ -type }{ -value }{ explain_content( ) }' ). + ri_html->add( lo_toolbar->render( ) ). + ri_html->add( '
' ). - ro_html->add( '
' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). ENDMETHOD. diff --git a/src/ui/db/zcl_abapgit_gui_page_db_dis.clas.abap b/src/ui/db/zcl_abapgit_gui_page_db_dis.clas.abap index 471ea2527..979ce8e64 100644 --- a/src/ui/db/zcl_abapgit_gui_page_db_dis.clas.abap +++ b/src/ui/db/zcl_abapgit_gui_page_db_dis.clas.abap @@ -57,20 +57,20 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_DB_DIS IMPLEMENTATION. lv_action = zcl_abapgit_html_action_utils=>dbkey_encode( ls_action ). lv_data = lo_highlighter->process_line( zcl_abapgit_xml_pretty=>print( lv_data ) ). - CREATE OBJECT ro_html. + CREATE OBJECT ri_html TYPE zcl_abapgit_html. CREATE OBJECT lo_toolbar. lo_toolbar->add( iv_act = |{ zif_abapgit_definitions=>c_action-db_edit }?{ lv_action }| iv_txt = 'Edit' ) ##NO_TEXT. - ro_html->add( '
' ). - ro_html->add( '
' ). - ro_html->add( render_record_banner( ms_key ) ). - ro_html->add( '' ). - ro_html->add( lo_toolbar->render( iv_right = abap_true ) ). - ro_html->add( '
' ). + ri_html->add( '
' ). + ri_html->add( '
' ). + ri_html->add( render_record_banner( ms_key ) ). + ri_html->add( '' ). + ri_html->add( lo_toolbar->render( iv_right = abap_true ) ). + ri_html->add( '
' ). - ro_html->add( |
{ lv_data }
| ). - ro_html->add( '
' ). + ri_html->add( |
{ lv_data }
| ). + ri_html->add( '
' ). ENDMETHOD. diff --git a/src/ui/db/zcl_abapgit_gui_page_db_edit.clas.abap b/src/ui/db/zcl_abapgit_gui_page_db_edit.clas.abap index 0a3c137ac..d63425145 100644 --- a/src/ui/db/zcl_abapgit_gui_page_db_edit.clas.abap +++ b/src/ui/db/zcl_abapgit_gui_page_db_edit.clas.abap @@ -97,30 +97,30 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_DB_EDIT IMPLEMENTATION. lv_data = escape( val = zcl_abapgit_xml_pretty=>print( lv_data ) format = cl_abap_format=>e_html_attr ). - CREATE OBJECT ro_html. + CREATE OBJECT ri_html TYPE zcl_abapgit_html. CREATE OBJECT lo_toolbar. lo_toolbar->add( iv_act = 'submitFormById(''db_form'');' iv_txt = 'Save' iv_typ = zif_abapgit_html=>c_action_type-onclick iv_opt = zif_abapgit_html=>c_html_opt-strong ) ##NO_TEXT. - ro_html->add( '
' ). + ri_html->add( '
' ). " Banners & Toolbar - ro_html->add( '
' ). - ro_html->add( zcl_abapgit_gui_page_db_dis=>render_record_banner( ms_key ) ). - ro_html->add( '' ). - ro_html->add( lo_toolbar->render( iv_right = abap_true ) ). - ro_html->add( '
' ). + ri_html->add( '
' ). + ri_html->add( zcl_abapgit_gui_page_db_dis=>render_record_banner( ms_key ) ). + ri_html->add( '' ). + ri_html->add( lo_toolbar->render( iv_right = abap_true ) ). + ri_html->add( '
' ). " Form - ro_html->add( |
| ). - ro_html->add( || ). - ro_html->add( || ). - ro_html->add( || ). - ro_html->add( '
' ). + ri_html->add( |
| ). + ri_html->add( || ). + ri_html->add( || ). + ri_html->add( || ). + ri_html->add( '
' ). - ro_html->add( '
' ). "db_entry + ri_html->add( '
' ). "db_entry ENDMETHOD. diff --git a/src/ui/zcl_abapgit_gui_page.clas.abap b/src/ui/zcl_abapgit_gui_page.clas.abap index 223c0c7a2..4f3474266 100644 --- a/src/ui/zcl_abapgit_gui_page.clas.abap +++ b/src/ui/zcl_abapgit_gui_page.clas.abap @@ -15,16 +15,18 @@ CLASS zcl_abapgit_gui_page DEFINITION PUBLIC ABSTRACT TYPES: BEGIN OF ty_control, - page_title TYPE string, - page_menu TYPE REF TO zcl_abapgit_html_toolbar, - END OF ty_control. + page_title TYPE string, + page_menu TYPE REF TO zcl_abapgit_html_toolbar, + END OF ty_control . - DATA: ms_control TYPE ty_control. - - METHODS render_content ABSTRACT - RETURNING VALUE(ro_html) TYPE REF TO zcl_abapgit_html - RAISING zcx_abapgit_exception. + DATA ms_control TYPE ty_control . + METHODS render_content + ABSTRACT + RETURNING + VALUE(ri_html) TYPE REF TO zif_abapgit_html + RAISING + zcx_abapgit_exception . PRIVATE SECTION. DATA: mo_settings TYPE REF TO zcl_abapgit_settings, @@ -33,29 +35,29 @@ CLASS zcl_abapgit_gui_page DEFINITION PUBLIC ABSTRACT METHODS render_deferred_parts IMPORTING - ii_html TYPE REF TO zif_abapgit_html + ii_html TYPE REF TO zif_abapgit_html iv_part_category TYPE string RAISING zcx_abapgit_exception. METHODS html_head - RETURNING VALUE(ro_html) TYPE REF TO zcl_abapgit_html. + RETURNING VALUE(ri_html) TYPE REF TO zif_abapgit_html. METHODS title - RETURNING VALUE(ro_html) TYPE REF TO zcl_abapgit_html. + RETURNING VALUE(ri_html) TYPE REF TO zif_abapgit_html. METHODS footer - RETURNING VALUE(ro_html) TYPE REF TO zcl_abapgit_html. + RETURNING VALUE(ri_html) TYPE REF TO zif_abapgit_html. METHODS render_link_hints IMPORTING - io_html TYPE REF TO zcl_abapgit_html + ii_html TYPE REF TO zif_abapgit_html RAISING zcx_abapgit_exception. METHODS render_command_palettes IMPORTING - io_html TYPE REF TO zcl_abapgit_html + ii_html TYPE REF TO zif_abapgit_html RAISING zcx_abapgit_exception. @@ -124,72 +126,72 @@ CLASS ZCL_ABAPGIT_GUI_PAGE IMPLEMENTATION. METHOD footer. - CREATE OBJECT ro_html. + CREATE OBJECT ri_html TYPE zcl_abapgit_html. - ro_html->add( '' ). "#EC NOTEXT ENDMETHOD. METHOD html_head. - CREATE OBJECT ro_html. + CREATE OBJECT ri_html TYPE zcl_abapgit_html. - ro_html->add( '' ). "#EC NOTEXT + ri_html->add( '' ). "#EC NOTEXT - ro_html->add( '' ). "#EC NOTEXT - ro_html->add( '' ). "#EC NOTEXT + ri_html->add( '' ). "#EC NOTEXT + ri_html->add( '' ). "#EC NOTEXT - ro_html->add( 'abapGit' ). "#EC NOTEXT - ro_html->add( '' ). - ro_html->add( '' ). + ri_html->add( 'abapGit' ). "#EC NOTEXT + ri_html->add( '' ). + ri_html->add( '' ). " Themes - ro_html->add( '' ). " Theme basis + ri_html->add( '' ). " Theme basis CASE mo_settings->get_ui_theme( ). WHEN zcl_abapgit_settings=>c_ui_theme-dark. - ro_html->add( '' ). + ri_html->add( '' ). WHEN zcl_abapgit_settings=>c_ui_theme-belize. - ro_html->add( '' ). + ri_html->add( '' ). ENDCASE. - ro_html->add( '' ). "#EC NOTEXT + ri_html->add( '' ). "#EC NOTEXT CASE mo_settings->get_icon_scaling( ). " Enforce icon scaling WHEN mo_settings->c_icon_scaling-large. - ro_html->add( '' ). + ri_html->add( '' ). WHEN mo_settings->c_icon_scaling-small. - ro_html->add( '' ). + ri_html->add( '' ). ENDCASE. - ro_html->add( '' ). "#EC NOTEXT + ri_html->add( '' ). "#EC NOTEXT ENDMETHOD. METHOD render_command_palettes. - io_html->add( 'var gGoRepoPalette = new CommandPalette(enumerateTocAllRepos, {' ). - io_html->add( ' toggleKey: "F2",' ). - io_html->add( ' hotkeyDescription: "Go to repo ..."' ). - io_html->add( '});' ). + ii_html->add( 'var gGoRepoPalette = new CommandPalette(enumerateTocAllRepos, {' ). + ii_html->add( ' toggleKey: "F2",' ). + ii_html->add( ' hotkeyDescription: "Go to repo ..."' ). + ii_html->add( '});' ). - io_html->add( 'var gCommandPalette = new CommandPalette(enumerateToolbarActions, {' ). - io_html->add( ' toggleKey: "F1",' ). - io_html->add( ' hotkeyDescription: "Command ..."' ). - io_html->add( '});' ). + ii_html->add( 'var gCommandPalette = new CommandPalette(enumerateToolbarActions, {' ). + ii_html->add( ' toggleKey: "F1",' ). + ii_html->add( ' hotkeyDescription: "Command ..."' ). + ii_html->add( '});' ). ENDMETHOD. @@ -254,9 +256,9 @@ CLASS ZCL_ABAPGIT_GUI_PAGE IMPLEMENTATION. IF mo_settings->get_link_hints_enabled( ) = abap_true AND lv_link_hint_key IS NOT INITIAL. - io_html->add( |activateLinkHints("{ lv_link_hint_key }");| ). - io_html->add( |setInitialFocusWithQuerySelector('a span', true);| ). - io_html->add( |enableArrowListNavigation();| ). + ii_html->add( |activateLinkHints("{ lv_link_hint_key }");| ). + ii_html->add( |setInitialFocusWithQuerySelector('a span', true);| ). + ii_html->add( |enableArrowListNavigation();| ). ENDIF. @@ -279,27 +281,27 @@ CLASS ZCL_ABAPGIT_GUI_PAGE IMPLEMENTATION. METHOD title. - CREATE OBJECT ro_html. + CREATE OBJECT ri_html TYPE zcl_abapgit_html. - ro_html->add( '' ). "#EC NOTEXT + ri_html->add( '' ). "#EC NOTEXT + ri_html->add( '' ). "#EC NOTEXT ENDMETHOD. @@ -348,7 +350,7 @@ CLASS ZCL_ABAPGIT_GUI_PAGE IMPLEMENTATION. METHOD zif_abapgit_gui_renderable~render. - DATA: lo_script TYPE REF TO zcl_abapgit_html. + DATA: li_script TYPE REF TO zif_abapgit_html. gui_services( )->register_event_handler( me ). @@ -373,11 +375,11 @@ CLASS ZCL_ABAPGIT_GUI_PAGE IMPLEMENTATION. ri_html->add( footer( ) ). ri_html->add( '' ). "#EC NOTEXT - lo_script = scripts( ). + li_script = scripts( ). - IF lo_script IS BOUND AND lo_script->is_empty( ) = abap_false. + IF li_script IS BOUND AND li_script->is_empty( ) = abap_false. ri_html->add( '' ). ENDIF. diff --git a/src/ui/zcl_abapgit_gui_page_bkg.clas.abap b/src/ui/zcl_abapgit_gui_page_bkg.clas.abap index 8a51e49f1..26694ded5 100644 --- a/src/ui/zcl_abapgit_gui_page_bkg.clas.abap +++ b/src/ui/zcl_abapgit_gui_page_bkg.clas.abap @@ -203,9 +203,9 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_BKG IMPLEMENTATION. METHOD render_content. - CREATE OBJECT ro_html. + CREATE OBJECT ri_html TYPE zcl_abapgit_html. - ro_html->add( render( ) ). + ri_html->add( render( ) ). ENDMETHOD. diff --git a/src/ui/zcl_abapgit_gui_page_bkg_run.clas.abap b/src/ui/zcl_abapgit_gui_page_bkg_run.clas.abap index 473b63ed2..6564ea9ba 100644 --- a/src/ui/zcl_abapgit_gui_page_bkg_run.clas.abap +++ b/src/ui/zcl_abapgit_gui_page_bkg_run.clas.abap @@ -38,13 +38,13 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_BKG_RUN IMPLEMENTATION. run( ). - CREATE OBJECT ro_html. + CREATE OBJECT ri_html TYPE zcl_abapgit_html. - ro_html->add( '
' ). + ri_html->add( '
' ). LOOP AT mt_text INTO lv_text. - ro_html->add( '
' && lv_text && '

' ). + ri_html->add( '
' && lv_text && '

' ). ENDLOOP. - ro_html->add( '
' ). + ri_html->add( '
' ). ENDMETHOD. diff --git a/src/ui/zcl_abapgit_gui_page_boverview.clas.abap b/src/ui/zcl_abapgit_gui_page_boverview.clas.abap index dcf060c4d..237abe31b 100644 --- a/src/ui/zcl_abapgit_gui_page_boverview.clas.abap +++ b/src/ui/zcl_abapgit_gui_page_boverview.clas.abap @@ -364,11 +364,11 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_BOVERVIEW IMPLEMENTATION. METHOD render_content. - CREATE OBJECT ro_html. + CREATE OBJECT ri_html TYPE zcl_abapgit_html. - ro_html->add( '
' ). - ro_html->add( body( ) ). - ro_html->add( '
' ). + ri_html->add( '
' ). + ri_html->add( body( ) ). + ri_html->add( '
' ). ENDMETHOD. diff --git a/src/ui/zcl_abapgit_gui_page_code_insp.clas.abap b/src/ui/zcl_abapgit_gui_page_code_insp.clas.abap index caeb7f4a4..41726cda8 100644 --- a/src/ui/zcl_abapgit_gui_page_code_insp.clas.abap +++ b/src/ui/zcl_abapgit_gui_page_code_insp.clas.abap @@ -185,30 +185,30 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_CODE_INSP IMPLEMENTATION. METHOD render_content. - CREATE OBJECT ro_html. + CREATE OBJECT ri_html TYPE zcl_abapgit_html. IF mv_check_variant IS INITIAL. - ro_html->add( zcl_abapgit_gui_chunk_lib=>render_error( iv_error = 'No check variant supplied.' ) ). + ri_html->add( zcl_abapgit_gui_chunk_lib=>render_error( iv_error = 'No check variant supplied.' ) ). RETURN. ENDIF. gui_services( )->get_hotkeys_ctl( )->register_hotkeys( me ). - ro_html->add( '
' ). - ro_html->add( |Code inspector check variant: { mv_check_variant }| ). - ro_html->add( |
{ + ri_html->add( '
' ). + ri_html->add( |Code inspector check variant: { mv_check_variant }| ). + ri_html->add( |
{ zcl_abapgit_html=>icon( 'box/grey70' ) }{ mo_repo->get_package( ) }
| ). - ro_html->add( '
' ). + ri_html->add( '
' ). IF lines( mt_result ) = 0. - ro_html->add( '
' ). - ro_html->add( zcl_abapgit_html=>icon( 'check' ) ). - ro_html->add( 'No code inspector findings' ). - ro_html->add( '
' ). + ri_html->add( '
' ). + ri_html->add( zcl_abapgit_html=>icon( 'check' ) ). + ri_html->add( 'No code inspector findings' ). + ri_html->add( '
' ). ELSE. render_result( - io_html = ro_html + ii_html = ri_html it_result = mt_result ). ENDIF. diff --git a/src/ui/zcl_abapgit_gui_page_codi_base.clas.abap b/src/ui/zcl_abapgit_gui_page_codi_base.clas.abap index 868392610..6dc4ef676 100644 --- a/src/ui/zcl_abapgit_gui_page_codi_base.clas.abap +++ b/src/ui/zcl_abapgit_gui_page_codi_base.clas.abap @@ -11,11 +11,11 @@ CLASS zcl_abapgit_gui_page_codi_base DEFINITION PUBLIC ABSTRACT INHERITING FROM METHODS render_result IMPORTING - !io_html TYPE REF TO zcl_abapgit_html + !ii_html TYPE REF TO zif_abapgit_html !it_result TYPE scit_alvlist . METHODS render_result_line IMPORTING - !io_html TYPE REF TO zcl_abapgit_html + !ii_html TYPE REF TO zif_abapgit_html !is_result TYPE scir_alvlist . METHODS build_nav_link IMPORTING @@ -128,21 +128,21 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_CODI_BASE IMPLEMENTATION. CONSTANTS: lc_limit TYPE i VALUE 500. FIELD-SYMBOLS: TYPE scir_alvlist. - io_html->add( '
' ). + ii_html->add( '
' ). LOOP AT it_result ASSIGNING TO lc_limit. render_result_line( - io_html = io_html + ii_html = ii_html is_result = ). ENDLOOP. - io_html->add( '
' ). + ii_html->add( '
' ). IF lines( it_result ) > lc_limit. - io_html->add( '
' ). - io_html->add( zcl_abapgit_html=>icon( 'exclamation-triangle' ) ). - io_html->add( |Only first { lc_limit } findings shown in list!| ). - io_html->add( '
' ). + ii_html->add( '
' ). + ii_html->add( zcl_abapgit_html=>icon( 'exclamation-triangle' ) ). + ii_html->add( |Only first { lc_limit } findings shown in list!| ). + ii_html->add( '
' ). ENDIF. ENDMETHOD. @@ -175,13 +175,13 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_CODI_BASE IMPLEMENTATION. ENDIF. lv_obj_txt = |{ lv_obj_txt } [ @{ zcl_abapgit_convert=>alpha_output( is_result-line ) } ]|. - io_html->add( |
  • | ). - io_html->add_a( + ii_html->add( |
  • | ). + ii_html->add_a( iv_txt = lv_obj_txt iv_act = build_nav_link( is_result ) iv_typ = zif_abapgit_html=>c_action_type-sapevent ). - io_html->add( |{ lv_msg }| ). - io_html->add( '
    • ' ). + ii_html->add( |{ lv_msg }| ). + ii_html->add( '' ). ENDMETHOD. diff --git a/src/ui/zcl_abapgit_gui_page_commit.clas.abap b/src/ui/zcl_abapgit_gui_page_commit.clas.abap index 45c851921..895210c79 100644 --- a/src/ui/zcl_abapgit_gui_page_commit.clas.abap +++ b/src/ui/zcl_abapgit_gui_page_commit.clas.abap @@ -239,18 +239,18 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_COMMIT IMPLEMENTATION. METHOD render_content. - CREATE OBJECT ro_html. + CREATE OBJECT ri_html TYPE zcl_abapgit_html. - ro_html->add( '
    ' ). - ro_html->add( zcl_abapgit_gui_chunk_lib=>render_repo_top( + ri_html->add( '
    ' ). + ri_html->add( zcl_abapgit_gui_chunk_lib=>render_repo_top( io_repo = mo_repo iv_show_package = abap_false iv_branch = mo_repo->get_branch_name( ) ) ). - ro_html->add( render_menu( ) ). - ro_html->add( render_form( ) ). - ro_html->add( render_stage( ) ). - ro_html->add( '
    ' ). + ri_html->add( render_menu( ) ). + ri_html->add( render_form( ) ). + ri_html->add( render_stage( ) ). + ri_html->add( '
    ' ). register_deferred_script( render_scripts( ) ). diff --git a/src/ui/zcl_abapgit_gui_page_debuginfo.clas.abap b/src/ui/zcl_abapgit_gui_page_debuginfo.clas.abap index d73b23e2a..e4793df45 100644 --- a/src/ui/zcl_abapgit_gui_page_debuginfo.clas.abap +++ b/src/ui/zcl_abapgit_gui_page_debuginfo.clas.abap @@ -56,15 +56,15 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_DEBUGINFO IMPLEMENTATION. METHOD render_content. - CREATE OBJECT ro_html. + CREATE OBJECT ri_html TYPE zcl_abapgit_html. - ro_html->add( '
    ' ). - ro_html->add( render_debug_info( ) ). - ro_html->add( '
    ' ). + ri_html->add( '
    ' ). + ri_html->add( render_debug_info( ) ). + ri_html->add( '
    ' ). - ro_html->add( '
    ' ). - ro_html->add( render_supported_object_types( ) ). - ro_html->add( '
    ' ). + ri_html->add( '
    ' ). + ri_html->add( render_supported_object_types( ) ). + ri_html->add( '
    ' ). register_deferred_script( render_scripts( ) ). diff --git a/src/ui/zcl_abapgit_gui_page_diff.clas.abap b/src/ui/zcl_abapgit_gui_page_diff.clas.abap index 79c2d698b..4bd1de152 100644 --- a/src/ui/zcl_abapgit_gui_page_diff.clas.abap +++ b/src/ui/zcl_abapgit_gui_page_diff.clas.abap @@ -565,23 +565,23 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_DIFF IMPLEMENTATION. li_progress TYPE REF TO zif_abapgit_progress. - CREATE OBJECT ro_html. + CREATE OBJECT ri_html TYPE zcl_abapgit_html. li_progress = zcl_abapgit_progress=>get_instance( lines( mt_diff_files ) ). - ro_html->add( |
    | ). - ro_html->add( zcl_abapgit_gui_chunk_lib=>render_js_error_banner( ) ). + ri_html->add( |
    | ). + ri_html->add( zcl_abapgit_gui_chunk_lib=>render_js_error_banner( ) ). LOOP AT mt_diff_files INTO ls_diff_file. li_progress->show( iv_current = sy-tabix iv_text = |Render Diff - { ls_diff_file-filename }| ). - ro_html->add( render_diff( ls_diff_file ) ). + ri_html->add( render_diff( ls_diff_file ) ). ENDLOOP. IF sy-subrc <> 0. - ro_html->add( |No more diffs| ). + ri_html->add( |No more diffs| ). ENDIF. - ro_html->add( '
    ' ). + ri_html->add( '
    ' ). register_deferred_script( render_scripts( ) ). diff --git a/src/ui/zcl_abapgit_gui_page_main.clas.abap b/src/ui/zcl_abapgit_gui_page_main.clas.abap index a34d5ef09..510cfabb9 100644 --- a/src/ui/zcl_abapgit_gui_page_main.clas.abap +++ b/src/ui/zcl_abapgit_gui_page_main.clas.abap @@ -33,13 +33,13 @@ CLASS zcl_abapgit_gui_page_main DEFINITION RAISING zcx_abapgit_exception, render_toc IMPORTING it_repo_list TYPE zif_abapgit_definitions=>ty_repo_ref_tt - RETURNING VALUE(ro_html) TYPE REF TO zcl_abapgit_html + RETURNING VALUE(ri_html) TYPE REF TO zif_abapgit_html RAISING zcx_abapgit_exception, build_main_menu RETURNING VALUE(ro_menu) TYPE REF TO zcl_abapgit_html_toolbar, render_repo IMPORTING io_repo TYPE REF TO zcl_abapgit_repo - RETURNING VALUE(ro_html) TYPE REF TO zcl_abapgit_html + RETURNING VALUE(ri_html) TYPE REF TO zif_abapgit_html RAISING zcx_abapgit_exception. ENDCLASS. @@ -112,24 +112,24 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_MAIN IMPLEMENTATION. retrieve_active_repo( ). " Get and validate key of user default repo - CREATE OBJECT ro_html. + CREATE OBJECT ri_html TYPE zcl_abapgit_html. gui_services( )->get_hotkeys_ctl( )->register_hotkeys( me ). TRY. lt_repos = zcl_abapgit_repo_srv=>get_instance( )->list( ). CATCH zcx_abapgit_exception INTO lx_error. - ro_html->add( zcl_abapgit_gui_chunk_lib=>render_error( ix_error = lx_error ) ). + ri_html->add( zcl_abapgit_gui_chunk_lib=>render_error( ix_error = lx_error ) ). RETURN. ENDTRY. - ro_html->add( render_toc( lt_repos ) ). + ri_html->add( render_toc( lt_repos ) ). IF mv_show IS INITIAL OR lines( lt_repos ) = 0. CREATE OBJECT li_tutorial TYPE zcl_abapgit_gui_view_tutorial. - ro_html->add( li_tutorial->render( ) ). + ri_html->add( li_tutorial->render( ) ). ELSE. lo_repo = zcl_abapgit_repo_srv=>get_instance( )->get( mv_show ). - ro_html->add( render_repo( lo_repo ) ). + ri_html->add( render_repo( lo_repo ) ). ENDIF. ENDMETHOD. @@ -139,22 +139,22 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_MAIN IMPLEMENTATION. DATA lo_news TYPE REF TO zcl_abapgit_news. - CREATE OBJECT ro_html. + CREATE OBJECT ri_html TYPE zcl_abapgit_html. lo_news = zcl_abapgit_news=>create( io_repo ). - ro_html->add( |
    | ). - ro_html->add( zcl_abapgit_gui_chunk_lib=>render_repo_top( + ri_html->add( |
    | ). + ri_html->add( zcl_abapgit_gui_chunk_lib=>render_repo_top( io_repo = io_repo io_news = lo_news iv_interactive_branch = abap_true ) ). - ro_html->add( zcl_abapgit_gui_chunk_lib=>render_news( io_news = lo_news ) ). + ri_html->add( zcl_abapgit_gui_chunk_lib=>render_news( io_news = lo_news ) ). IF mo_repo_content IS BOUND. - ro_html->add( mo_repo_content->zif_abapgit_gui_renderable~render( ) ). + ri_html->add( mo_repo_content->zif_abapgit_gui_renderable~render( ) ). ENDIF. - ro_html->add( '
    ' ). + ri_html->add( '
    ' ). ENDMETHOD. @@ -172,7 +172,7 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_MAIN IMPLEMENTATION. lv_repo_title TYPE string. - CREATE OBJECT ro_html. + CREATE OBJECT ri_html TYPE zcl_abapgit_html. CREATE OBJECT lo_favbar. CREATE OBJECT lo_allbar EXPORTING iv_id = 'toc-all-repos'. CREATE OBJECT lo_pback. @@ -219,43 +219,43 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_MAIN IMPLEMENTATION. ENDLOOP. " Render HTML - ro_html->add( '
    ' ) ##NO_TEXT. " TODO refactor html & css - ro_html->add( '
    ' ) ##NO_TEXT. - ro_html->add( '
    ' ) ##NO_TEXT. + ri_html->add( '
    ' ) ##NO_TEXT. " TODO refactor html & css + ri_html->add( '
    ' ) ##NO_TEXT. + ri_html->add( '
    ' ) ##NO_TEXT. ********************************************************************** - ro_html->add( '' ). - ro_html->add( |' ). + ri_html->add( '
    { + ri_html->add( '' ). + ri_html->add( || ). - ro_html->add( '' ). + ri_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '
    { zcl_abapgit_html=>icon( iv_name = 'star/blue' iv_hint = 'Favorites' ) }' ). " Maximize width + ri_html->add( '' ). " Maximize width IF lo_favbar->count( ) > 0. - ro_html->add( lo_favbar->render( iv_sort = abap_true ) ). + ri_html->add( lo_favbar->render( iv_sort = abap_true ) ). ELSE. - ro_html->add( |No favorites so far. For more info please check { + ri_html->add( |No favorites so far. For more info please check { zcl_abapgit_html=>a( iv_txt = 'tutorial' iv_act = zif_abapgit_definitions=>c_action-go_tutorial ) }| ). ENDIF. - ro_html->add( '' ). - ro_html->add( lo_allbar->render_as_droplist( + ri_html->add( '' ). + ri_html->add( lo_allbar->render_as_droplist( iv_label = zcl_abapgit_html=>icon( iv_name = 'bars/blue' ) iv_action = c_actions-overview iv_right = abap_true iv_sort = abap_true ) ). - ro_html->add( '
    ' ). + ri_html->add( '
    ' ). ********************************************************************** - ro_html->add( '
    ' ). - ro_html->add( '
    ' ). - ro_html->add( '
    ' ). + ri_html->add( '
    ' ). + ri_html->add( '
    ' ). + ri_html->add( '
    ' ). ENDMETHOD. diff --git a/src/ui/zcl_abapgit_gui_page_merge.clas.abap b/src/ui/zcl_abapgit_gui_page_merge.clas.abap index 59c5362af..2b0247cc0 100644 --- a/src/ui/zcl_abapgit_gui_page_merge.clas.abap +++ b/src/ui/zcl_abapgit_gui_page_merge.clas.abap @@ -32,7 +32,7 @@ CLASS zcl_abapgit_gui_page_merge DEFINITION METHODS show_file IMPORTING !it_expanded TYPE zif_abapgit_definitions=>ty_expanded_tt - !io_html TYPE REF TO zcl_abapgit_html + !ii_html TYPE REF TO zif_abapgit_html !is_file TYPE zif_abapgit_definitions=>ty_expanded !is_result TYPE zif_abapgit_definitions=>ty_expanded . METHODS build_menu @@ -96,33 +96,33 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_MERGE IMPLEMENTATION. "If now exists no conflicts anymore, conflicts button should disappear ms_control-page_menu = build_menu( mo_merge->has_conflicts( ) ). - CREATE OBJECT ro_html. + CREATE OBJECT ri_html TYPE zcl_abapgit_html. - ro_html->add( '
    ' ). - ro_html->add( zcl_abapgit_gui_chunk_lib=>render_repo_top( + ri_html->add( '
    ' ). + ri_html->add( zcl_abapgit_gui_chunk_lib=>render_repo_top( io_repo = mo_repo iv_show_package = abap_false iv_show_branch = abap_false ) ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '
    Source' ). - ro_html->add( ls_merge-source-name ). - ro_html->add( '
    Target' ). - ro_html->add( ls_merge-target-name ). - ro_html->add( '
    Ancestor' ). - ro_html->add( ls_merge-common-commit ). - ro_html->add( '
    ' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '
    Source' ). + ri_html->add( ls_merge-source-name ). + ri_html->add( '
    Target' ). + ri_html->add( ls_merge-target-name ). + ri_html->add( '
    Ancestor' ). + ri_html->add( ls_merge-common-commit ). + ri_html->add( '
    ' ). - ro_html->add( '
    ' ). + ri_html->add( '
    ' ). APPEND LINES OF ls_merge-stree TO lt_files. APPEND LINES OF ls_merge-ttree TO lt_files. @@ -130,47 +130,47 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_MERGE IMPLEMENTATION. SORT lt_files BY path DESCENDING name ASCENDING. DELETE ADJACENT DUPLICATES FROM lt_files COMPARING path name. - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '' ). + ri_html->add( '
    SourceTargetAncestorResult
    ' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '' ). LOOP AT lt_files ASSIGNING . CLEAR ls_result. READ TABLE ls_merge-result INTO ls_result WITH KEY path = -path name = -name. - ro_html->add( '' ). + ri_html->add( '' ). show_file( it_expanded = ls_merge-stree - io_html = ro_html + ii_html = ri_html is_file = is_result = ls_result ). show_file( it_expanded = ls_merge-ttree - io_html = ro_html + ii_html = ri_html is_file = is_result = ls_result ). show_file( it_expanded = ls_merge-ctree - io_html = ro_html + ii_html = ri_html is_file = is_result = ls_result ). show_file( it_expanded = ls_merge-result - io_html = ro_html + ii_html = ri_html is_file = is_result = ls_result ). - ro_html->add( '' ). + ri_html->add( '' ). ENDLOOP. - ro_html->add( '
    SourceTargetAncestorResult
    ' ). - ro_html->add( '
    ' ). - ro_html->add( '' ). - ro_html->add( ls_merge-conflict ). - ro_html->add( '' ). - ro_html->add( '
    ' ). + ri_html->add( '' ). + ri_html->add( '
    ' ). + ri_html->add( '' ). + ri_html->add( ls_merge-conflict ). + ri_html->add( '' ). + ri_html->add( '
    ' ). ENDMETHOD. @@ -186,12 +186,12 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_MERGE IMPLEMENTATION. name = is_file-name. IF sy-subrc = 0. IF -sha1 = is_result-sha1. - io_html->add( |{ -path }{ -name }{ -sha1(7) }| ). + ii_html->add( |{ -path }{ -name }{ -sha1(7) }| ). ELSE. - io_html->add( |{ -path }{ -name }{ -sha1(7) }| ). + ii_html->add( |{ -path }{ -name }{ -sha1(7) }| ). ENDIF. ELSE. - io_html->add( '' ). + ii_html->add( '' ). ENDIF. ENDMETHOD. diff --git a/src/ui/zcl_abapgit_gui_page_merge_res.clas.abap b/src/ui/zcl_abapgit_gui_page_merge_res.clas.abap index 2516ae69a..af160a27a 100644 --- a/src/ui/zcl_abapgit_gui_page_merge_res.clas.abap +++ b/src/ui/zcl_abapgit_gui_page_merge_res.clas.abap @@ -250,10 +250,10 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_MERGE_RES IMPLEMENTATION. zcx_abapgit_exception=>raise( 'no conflict found' ). ENDIF. - CREATE OBJECT ro_html. - ro_html->add( |
    | ). - ro_html->add( render_diff( ms_diff_file ) ). - ro_html->add( '
    ' ). + CREATE OBJECT ri_html TYPE zcl_abapgit_html. + ri_html->add( |
    | ). + ri_html->add( render_diff( ms_diff_file ) ). + ri_html->add( '
    ' ). ENDMETHOD. diff --git a/src/ui/zcl_abapgit_gui_page_patch.clas.abap b/src/ui/zcl_abapgit_gui_page_patch.clas.abap index fe45b8788..37634de27 100644 --- a/src/ui/zcl_abapgit_gui_page_patch.clas.abap +++ b/src/ui/zcl_abapgit_gui_page_patch.clas.abap @@ -581,7 +581,7 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_PATCH IMPLEMENTATION. ENDIF. gui_services( )->get_hotkeys_ctl( )->register_hotkeys( me ). - ro_html = super->render_content( ). + ri_html = super->render_content( ). register_deferred_script( render_scripts( ) ). diff --git a/src/ui/zcl_abapgit_gui_page_repo_over.clas.abap b/src/ui/zcl_abapgit_gui_page_repo_over.clas.abap index 85b637d3b..a1887136a 100644 --- a/src/ui/zcl_abapgit_gui_page_repo_over.clas.abap +++ b/src/ui/zcl_abapgit_gui_page_repo_over.clas.abap @@ -70,21 +70,21 @@ CLASS zcl_abapgit_gui_page_repo_over DEFINITION render_table_header IMPORTING - io_html TYPE REF TO zcl_abapgit_html, + ii_html TYPE REF TO zif_abapgit_html, render_table IMPORTING - io_html TYPE REF TO zcl_abapgit_html + ii_html TYPE REF TO zif_abapgit_html it_overview TYPE tty_overview, render_table_body IMPORTING - io_html TYPE REF TO zcl_abapgit_html + ii_html TYPE REF TO zif_abapgit_html it_overview TYPE tty_overview, render_header_bar IMPORTING - io_html TYPE REF TO zcl_abapgit_html, + ii_html TYPE REF TO zif_abapgit_html, apply_order_by CHANGING ct_overview TYPE tty_overview, @@ -240,10 +240,10 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_REPO_OVER IMPLEMENTATION. apply_filter( CHANGING ct_overview = lt_overview ). - CREATE OBJECT ro_html. + CREATE OBJECT ri_html TYPE zcl_abapgit_html. - render_header_bar( ro_html ). - render_table( io_html = ro_html + render_header_bar( ri_html ). + render_table( ii_html = ri_html it_overview = lt_overview ). register_deferred_script( render_scripts( ) ). @@ -253,23 +253,23 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_REPO_OVER IMPLEMENTATION. METHOD render_header_bar. - io_html->add( |
    | ). + ii_html->add( |
    | ). - io_html->add( |
    | ). + ii_html->add( || ). - io_html->add( render_text_input( + ii_html->add( render_text_input( iv_name = |filter| iv_label = |Filter: | iv_value = mv_filter ) ). - io_html->add( || ). - io_html->add( |
    | ). + ii_html->add( || ). + ii_html->add( || ). - io_html->add( zcl_abapgit_html=>a( + ii_html->add( zcl_abapgit_html=>a( iv_txt = 'Toggle detail' iv_act = |gHelper.toggleRepoListDetail()| iv_typ = zif_abapgit_html=>c_action_type-onclick ) ). - io_html->add( |
    | ). + ii_html->add( |
    | ). ENDMETHOD. @@ -287,15 +287,15 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_REPO_OVER IMPLEMENTATION. METHOD render_table. - io_html->add( |
    | ). - io_html->add( || ). + ii_html->add( |
    | ). + ii_html->add( |
    | ). - render_table_header( io_html ). - render_table_body( io_html = io_html + render_table_header( ii_html ). + render_table_body( ii_html = ii_html it_overview = it_overview ). - io_html->add( |
    | ). - io_html->add( |
    | ). + ii_html->add( || ). + ii_html->add( |
    | ). ENDMETHOD. @@ -308,7 +308,7 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_REPO_OVER IMPLEMENTATION. FIELD-SYMBOLS: LIKE LINE OF it_overview. - io_html->add( '' ). + ii_html->add( '' ). LOOP AT it_overview ASSIGNING . @@ -324,38 +324,38 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_REPO_OVER IMPLEMENTATION. lv_favorite_icon = 'star/grey'. ENDIF. - io_html->add( || ). - io_html->add( || ). - io_html->add_a( iv_act = |{ zif_abapgit_definitions=>c_action-repo_toggle_fav }?{ -key }| + ii_html->add( || ). + ii_html->add( || ). + ii_html->add_a( iv_act = |{ zif_abapgit_definitions=>c_action-repo_toggle_fav }?{ -key }| iv_txt = zcl_abapgit_html=>icon( iv_name = lv_favorite_icon iv_class = 'pad-sides' iv_hint = 'Click to toggle favorite' ) ). - io_html->add( || ). - io_html->add( |{ zcl_abapgit_html=>icon( lv_type_icon ) }| ). + ii_html->add( || ). + ii_html->add( |{ zcl_abapgit_html=>icon( lv_type_icon ) }| ). - io_html->add( |{ zcl_abapgit_html=>a( iv_txt = -name + ii_html->add( |{ zcl_abapgit_html=>a( iv_txt = -name iv_act = |{ c_action-select }?{ -key }| ) }| ). IF -type = abap_false. - io_html->add( |{ io_html->a( iv_txt = -url + ii_html->add( |{ ii_html->a( iv_txt = -url iv_act = |{ zif_abapgit_definitions=>c_action-url }?| && |{ -url }| ) }| ). ELSE. - io_html->add( || ). + ii_html->add( || ). ENDIF. - io_html->add( |{ -package }| ). - io_html->add( |{ -branch }| ). - io_html->add( |{ -deserialized_by }| ). - io_html->add( |{ -deserialized_at }| ). - io_html->add( |{ -created_by }| ). - io_html->add( |{ -created_at }| ). - io_html->add( |{ -key }| ). - io_html->add( || ). + ii_html->add( |{ -package }| ). + ii_html->add( |{ -branch }| ). + ii_html->add( |{ -deserialized_by }| ). + ii_html->add( |{ -deserialized_at }| ). + ii_html->add( |{ -created_by }| ). + ii_html->add( |{ -created_at }| ). + ii_html->add( |{ -key }| ). + ii_html->add( || ). ENDLOOP. - io_html->add( || ). + ii_html->add( || ). ENDMETHOD. @@ -376,16 +376,16 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_REPO_OVER IMPLEMENTATION. _add_col( 'CREATED_AT /Created at /ro-detail /X' ). _add_col( 'KEY /Key /ro-detail / ' ). - io_html->add( || ). - io_html->add( || ). + ii_html->add( || ). + ii_html->add( || ). - io_html->add( zcl_abapgit_gui_chunk_lib=>render_order_by_header_cells( + ii_html->add( zcl_abapgit_gui_chunk_lib=>render_order_by_header_cells( it_col_spec = mt_col_spec iv_order_by = mv_order_by iv_order_descending = mv_order_descending ) ). - io_html->add( '' ). - io_html->add( '' ). + ii_html->add( '' ). + ii_html->add( '' ). ENDMETHOD. diff --git a/src/ui/zcl_abapgit_gui_page_repo_sett.clas.abap b/src/ui/zcl_abapgit_gui_page_repo_sett.clas.abap index 5cdcf6e2a..e4d1e37eb 100644 --- a/src/ui/zcl_abapgit_gui_page_repo_sett.clas.abap +++ b/src/ui/zcl_abapgit_gui_page_repo_sett.clas.abap @@ -23,10 +23,10 @@ CLASS zcl_abapgit_gui_page_repo_sett DEFINITION METHODS render_dot_abapgit IMPORTING - !io_html TYPE REF TO zcl_abapgit_html . + !ii_html TYPE REF TO zif_abapgit_html . METHODS render_local_settings IMPORTING - !io_html TYPE REF TO zcl_abapgit_html + !ii_html TYPE REF TO zif_abapgit_html RAISING zcx_abapgit_exception . METHODS save @@ -51,7 +51,7 @@ CLASS zcl_abapgit_gui_page_repo_sett DEFINITION VALUE(rt_post_fields) TYPE tihttpnvp . METHODS render_dot_abapgit_reqs IMPORTING - io_html TYPE REF TO zcl_abapgit_html + ii_html TYPE REF TO zif_abapgit_html it_requirements TYPE zif_abapgit_dot_abapgit=>ty_requirement_tt. METHODS render_table_row IMPORTING @@ -92,16 +92,16 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_REPO_SETT IMPLEMENTATION. METHOD render_content. - CREATE OBJECT ro_html. - ro_html->add( '
    ' ). - ro_html->add( |
    | ). + CREATE OBJECT ri_html TYPE zcl_abapgit_html. + ri_html->add( '
    ' ). + ri_html->add( || ). - render_dot_abapgit( ro_html ). - render_local_settings( ro_html ). + render_dot_abapgit( ri_html ). + render_local_settings( ri_html ). - ro_html->add( '' ). - ro_html->add( '' ). - ro_html->add( '
    ' ). + ri_html->add( '' ). + ri_html->add( '' ). + ri_html->add( '
    ' ). ENDMETHOD. @@ -123,8 +123,8 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_REPO_SETT IMPLEMENTATION. APPEND zif_abapgit_dot_abapgit=>c_folder_logic-full TO lt_folder_logic. APPEND zif_abapgit_dot_abapgit=>c_folder_logic-prefix TO lt_folder_logic. - io_html->add( '

    .abapgit.xml

    ' ). - io_html->add( '' ). + ii_html->add( '

    .abapgit.xml

    ' ). + ii_html->add( '
    ' ). SELECT SINGLE sptxt INTO lv_language FROM t002t WHERE spras = sy-langu AND sprsl = ls_dot-master_language. @@ -132,7 +132,7 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_REPO_SETT IMPLEMENTATION. lv_language = 'Unknown language. Check your settings.'. ENDIF. - io_html->add( render_table_row( + ii_html->add( render_table_row( iv_name = 'Master language' iv_value = |{ ls_dot-master_language } ({ lv_language })| ) ). @@ -152,12 +152,12 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_REPO_SETT IMPLEMENTATION. ENDLOOP. lv_select_html = lv_select_html && ''. - io_html->add( render_table_row( + ii_html->add( render_table_row( iv_name = 'Folder logic' iv_value = lv_select_html ) ). - io_html->add( render_table_row( + ii_html->add( render_table_row( iv_name = 'Starting folder' iv_value = || ) ). @@ -166,17 +166,17 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_REPO_SETT IMPLEMENTATION. lv_ignore = lv_ignore && && zif_abapgit_definitions=>c_newline. ENDLOOP. - io_html->add( render_table_row( + ii_html->add( render_table_row( iv_name = 'Ignore files' iv_value = || ) ). - io_html->add( '
    ' ). + ii_html->add( '' ). render_dot_abapgit_reqs( it_requirements = ls_dot-requirements - io_html = io_html ). + ii_html = ii_html ). ENDMETHOD. @@ -198,24 +198,24 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_REPO_SETT IMPLEMENTATION. ENDDO. ENDIF. - io_html->add( '

    Requirements

    ' ). - io_html->add( '' ). - io_html->add( '' ). + ii_html->add( '

    Requirements

    ' ). + ii_html->add( '
    Software ComponentMin ReleaseMin Patch
    ' ). + ii_html->add( '' ). LOOP AT lt_requirements ASSIGNING . lv_req_index = sy-tabix. - io_html->add( '' ). - io_html->add( |' ). + ii_html->add( || ). - io_html->add( || ). - io_html->add( || ). - io_html->add( '' ). + ii_html->add( '' ). ENDLOOP. - io_html->add( '
    Software ComponentMin ReleaseMin Patch
    add( '
    add( |add( |
    ' ). + ii_html->add( '' ). ENDMETHOD. @@ -227,10 +227,10 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_REPO_SETT IMPLEMENTATION. ls_settings = mo_repo->get_local_settings( ). - io_html->add( '

    Local settings

    ' ). - io_html->add( '' ). + ii_html->add( '

    Local settings

    ' ). + ii_html->add( '
    ' ). - io_html->add( render_table_row( + ii_html->add( render_table_row( iv_name = 'Display name' iv_value = || ) ). @@ -243,7 +243,7 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_REPO_SETT IMPLEMENTATION. lv_checked = | checked disabled|. ENDIF. ENDIF. - io_html->add( render_table_row( + ii_html->add( render_table_row( iv_name = 'Write protected' iv_value = || ) ). @@ -252,7 +252,7 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_REPO_SETT IMPLEMENTATION. IF ls_settings-ignore_subpackages = abap_true. lv_checked = | checked|. ENDIF. - io_html->add( render_table_row( + ii_html->add( render_table_row( iv_name = 'Ignore subpackages' iv_value = || ) ). @@ -261,12 +261,12 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_REPO_SETT IMPLEMENTATION. IF ls_settings-only_local_objects = abap_true. lv_checked = | checked|. ENDIF. - io_html->add( render_table_row( + ii_html->add( render_table_row( iv_name = 'Only local objects' iv_value = || ) ). - io_html->add( render_table_row( + ii_html->add( render_table_row( iv_name = 'Code inspector check variant' iv_value = || @@ -276,7 +276,7 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_REPO_SETT IMPLEMENTATION. IF ls_settings-block_commit = abap_true. lv_checked = | checked|. ENDIF. - io_html->add( render_table_row( + ii_html->add( render_table_row( iv_name = 'Block commit if code inspection has errors' iv_value = || ) ). @@ -285,12 +285,12 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_REPO_SETT IMPLEMENTATION. IF ls_settings-serialize_master_lang_only = abap_true. lv_checked = | checked|. ENDIF. - io_html->add( render_table_row( + ii_html->add( render_table_row( iv_name = 'Serialize master language only' iv_value = || ) ). - io_html->add( '
    ' ). + ii_html->add( '' ). ENDMETHOD. diff --git a/src/ui/zcl_abapgit_gui_page_settings.clas.abap b/src/ui/zcl_abapgit_gui_page_settings.clas.abap index 8311d0a26..a3794209c 100644 --- a/src/ui/zcl_abapgit_gui_page_settings.clas.abap +++ b/src/ui/zcl_abapgit_gui_page_settings.clas.abap @@ -392,33 +392,33 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_SETTINGS IMPLEMENTATION. METHOD render_content. - CREATE OBJECT ro_html. + CREATE OBJECT ri_html TYPE zcl_abapgit_html. read_settings( ). - ro_html->add( render_form_begin( ) ). - ro_html->add( render_section_begin( |Global settings| ) ). - ro_html->add( render_proxy( ) ). - ro_html->add( |
    | ). - ro_html->add( render_commit_msg( ) ). - ro_html->add( |
    | ). - ro_html->add( render_development_internals( ) ). - ro_html->add( render_section_end( ) ). - ro_html->add( render_section_begin( |User specific settings| ) ). - ro_html->add( render_start_up( ) ). - ro_html->add( render_max_lines( ) ). - ro_html->add( render_icon_scaling( ) ). - ro_html->add( render_ui_theme( ) ). - ro_html->add( |
    | ). - ro_html->add( render_adt_jump_enabled( ) ). - ro_html->add( |
    | ). - ro_html->add( render_parallel_proc( ) ). - ro_html->add( |
    | ). - ro_html->add( render_link_hints( ) ). - ro_html->add( |
    | ). - ro_html->add( render_hotkeys( ) ). - ro_html->add( render_section_end( ) ). - ro_html->add( render_form_end( ) ). + ri_html->add( render_form_begin( ) ). + ri_html->add( render_section_begin( |Global settings| ) ). + ri_html->add( render_proxy( ) ). + ri_html->add( |
    | ). + ri_html->add( render_commit_msg( ) ). + ri_html->add( |
    | ). + ri_html->add( render_development_internals( ) ). + ri_html->add( render_section_end( ) ). + ri_html->add( render_section_begin( |User specific settings| ) ). + ri_html->add( render_start_up( ) ). + ri_html->add( render_max_lines( ) ). + ri_html->add( render_icon_scaling( ) ). + ri_html->add( render_ui_theme( ) ). + ri_html->add( |
    | ). + ri_html->add( render_adt_jump_enabled( ) ). + ri_html->add( |
    | ). + ri_html->add( render_parallel_proc( ) ). + ri_html->add( |
    | ). + ri_html->add( render_link_hints( ) ). + ri_html->add( |
    | ). + ri_html->add( render_hotkeys( ) ). + ri_html->add( render_section_end( ) ). + ri_html->add( render_form_end( ) ). ENDMETHOD. diff --git a/src/ui/zcl_abapgit_gui_page_stage.clas.abap b/src/ui/zcl_abapgit_gui_page_stage.clas.abap index 02010057a..a253a6910 100644 --- a/src/ui/zcl_abapgit_gui_page_stage.clas.abap +++ b/src/ui/zcl_abapgit_gui_page_stage.clas.abap @@ -301,19 +301,19 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_STAGE IMPLEMENTATION. METHOD render_content. - CREATE OBJECT ro_html. + CREATE OBJECT ri_html TYPE zcl_abapgit_html. - ro_html->add( '
    ' ). - ro_html->add( zcl_abapgit_gui_chunk_lib=>render_repo_top( mo_repo ) ). - ro_html->add( zcl_abapgit_gui_chunk_lib=>render_js_error_banner( ) ). - ro_html->add( render_master_language_warning( ) ). + ri_html->add( '
    ' ). + ri_html->add( zcl_abapgit_gui_chunk_lib=>render_repo_top( mo_repo ) ). + ri_html->add( zcl_abapgit_gui_chunk_lib=>render_js_error_banner( ) ). + ri_html->add( render_master_language_warning( ) ). - ro_html->add( '
    ' ). - ro_html->add( render_actions( ) ). - ro_html->add( render_list( ) ). - ro_html->add( '
    ' ). + ri_html->add( '
    ' ). + ri_html->add( render_actions( ) ). + ri_html->add( render_list( ) ). + ri_html->add( '
    ' ). - ro_html->add( '
    ' ). + ri_html->add( '
    ' ). gui_services( )->get_hotkeys_ctl( )->register_hotkeys( me ). gui_services( )->get_html_parts( )->add_part( diff --git a/src/ui/zcl_abapgit_gui_page_syntax.clas.abap b/src/ui/zcl_abapgit_gui_page_syntax.clas.abap index 4fdac6665..b82b70a2e 100644 --- a/src/ui/zcl_abapgit_gui_page_syntax.clas.abap +++ b/src/ui/zcl_abapgit_gui_page_syntax.clas.abap @@ -66,19 +66,19 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_SYNTAX IMPLEMENTATION. METHOD render_content. - CREATE OBJECT ro_html. - ro_html->add( '
    ' ). + CREATE OBJECT ri_html TYPE zcl_abapgit_html. + ri_html->add( '
    ' ). IF lines( mt_result ) = 0. - ro_html->add( '
    ' ). - ro_html->add( zcl_abapgit_html=>icon( 'check' ) ). - ro_html->add( 'No syntax errors' ). + ri_html->add( '
    ' ). + ri_html->add( zcl_abapgit_html=>icon( 'check' ) ). + ri_html->add( 'No syntax errors' ). ELSE. - render_result( io_html = ro_html + render_result( ii_html = ri_html it_result = mt_result ). ENDIF. - ro_html->add( '
    ' ). + ri_html->add( '
    ' ). ENDMETHOD. diff --git a/src/ui/zcl_abapgit_gui_page_tag.clas.abap b/src/ui/zcl_abapgit_gui_page_tag.clas.abap index 56ddc08ed..71bdd8d01 100644 --- a/src/ui/zcl_abapgit_gui_page_tag.clas.abap +++ b/src/ui/zcl_abapgit_gui_page_tag.clas.abap @@ -192,12 +192,12 @@ CLASS ZCL_ABAPGIT_GUI_PAGE_TAG IMPLEMENTATION. METHOD render_content. - CREATE OBJECT ro_html. + CREATE OBJECT ri_html TYPE zcl_abapgit_html. - ro_html->add( '
    ' ). - ro_html->add( render_menu( ) ). - ro_html->add( render_form( ) ). - ro_html->add( '
    ' ). + ri_html->add( '
    ' ). + ri_html->add( render_menu( ) ). + ri_html->add( render_form( ) ). + ri_html->add( '
    ' ). register_deferred_script( render_scripts( ) ). diff --git a/src/ui/zcl_abapgit_hotkeys.clas.abap b/src/ui/zcl_abapgit_hotkeys.clas.abap index e6229a0db..98bceaf1e 100644 --- a/src/ui/zcl_abapgit_hotkeys.clas.abap +++ b/src/ui/zcl_abapgit_hotkeys.clas.abap @@ -35,62 +35,51 @@ CLASS zcl_abapgit_hotkeys DEFINITION PROTECTED SECTION. PRIVATE SECTION. - DATA mt_hotkey_providers TYPE TABLE OF REF TO zif_abapgit_gui_hotkeys. - - CLASS-DATA: - gv_hint_was_shown TYPE abap_bool, - gt_interface_implementations TYPE saboo_iimpt. - - CLASS-METHODS: - filter_relevant_classes - IMPORTING - it_classes TYPE seo_relkeys - RETURNING - VALUE(rt_classes) TYPE seo_relkeys, - - get_class_package - IMPORTING - iv_class_name TYPE seoclsname - RETURNING - VALUE(rv_package) TYPE devclass, - - get_referred_class_name - IMPORTING - io_ref TYPE any - RETURNING - VALUE(rv_name) TYPE seoclsname, - - get_hotkeys_by_class_name - IMPORTING - iv_class_name TYPE seoclsname - RETURNING - VALUE(rt_hotkeys) TYPE zif_abapgit_gui_hotkeys=>tty_hotkey_with_descr, - - get_hotkeys_from_global_intf - RETURNING - VALUE(rt_hotkeys) TYPE zif_abapgit_gui_hotkeys=>tty_hotkey_with_descr - RAISING - zcx_abapgit_exception, - - get_hotkeys_from_local_intf - RETURNING - VALUE(rt_hotkeys) TYPE zif_abapgit_gui_hotkeys=>tty_hotkey_with_descr - RAISING - zcx_abapgit_exception, - - get_local_intf_implementations - RETURNING - VALUE(rt_interface_implementations) TYPE saboo_iimpt - RAISING - zcx_abapgit_exception. - - METHODS - render_scripts - IMPORTING - it_hotkeys TYPE zif_abapgit_gui_hotkeys=>tty_hotkey_with_descr - RETURNING - VALUE(ro_html) TYPE REF TO zcl_abapgit_html. + DATA: + mt_hotkey_providers TYPE TABLE OF REF TO zif_abapgit_gui_hotkeys . + CLASS-DATA gv_hint_was_shown TYPE abap_bool . + CLASS-DATA gt_interface_implementations TYPE saboo_iimpt . + CLASS-METHODS filter_relevant_classes + IMPORTING + !it_classes TYPE seo_relkeys + RETURNING + VALUE(rt_classes) TYPE seo_relkeys . + CLASS-METHODS get_class_package + IMPORTING + !iv_class_name TYPE seoclsname + RETURNING + VALUE(rv_package) TYPE devclass . + CLASS-METHODS get_referred_class_name + IMPORTING + !io_ref TYPE any + RETURNING + VALUE(rv_name) TYPE seoclsname . + CLASS-METHODS get_hotkeys_by_class_name + IMPORTING + !iv_class_name TYPE seoclsname + RETURNING + VALUE(rt_hotkeys) TYPE zif_abapgit_gui_hotkeys=>tty_hotkey_with_descr . + CLASS-METHODS get_hotkeys_from_global_intf + RETURNING + VALUE(rt_hotkeys) TYPE zif_abapgit_gui_hotkeys=>tty_hotkey_with_descr + RAISING + zcx_abapgit_exception . + CLASS-METHODS get_hotkeys_from_local_intf + RETURNING + VALUE(rt_hotkeys) TYPE zif_abapgit_gui_hotkeys=>tty_hotkey_with_descr + RAISING + zcx_abapgit_exception . + CLASS-METHODS get_local_intf_implementations + RETURNING + VALUE(rt_interface_implementations) TYPE saboo_iimpt + RAISING + zcx_abapgit_exception . + METHODS render_scripts + IMPORTING + !it_hotkeys TYPE zif_abapgit_gui_hotkeys=>tty_hotkey_with_descr + RETURNING + VALUE(ri_html) TYPE REF TO zif_abapgit_html . ENDCLASS. @@ -282,9 +271,9 @@ CLASS ZCL_ABAPGIT_HOTKEYS IMPLEMENTATION. lv_json = lv_json && `}`. - CREATE OBJECT ro_html. - ro_html->zif_abapgit_html~set_title( cl_abap_typedescr=>describe_by_object_ref( me )->get_relative_name( ) ). - ro_html->add( |setKeyBindings({ lv_json });| ). + CREATE OBJECT ri_html TYPE zcl_abapgit_html. + ri_html->set_title( cl_abap_typedescr=>describe_by_object_ref( me )->get_relative_name( ) ). + ri_html->add( |setKeyBindings({ lv_json });| ). ENDMETHOD. @@ -349,9 +338,9 @@ CLASS ZCL_ABAPGIT_HOTKEYS IMPLEMENTATION. METHOD zif_abapgit_gui_renderable~render. DATA: - lv_hint TYPE string, - lt_registered_hotkeys TYPE zif_abapgit_gui_hotkeys=>tty_hotkey_with_descr, - lv_hotkey TYPE string. + lv_hint TYPE string, + lt_registered_hotkeys TYPE zif_abapgit_gui_hotkeys=>tty_hotkey_with_descr, + lv_hotkey TYPE string. FIELD-SYMBOLS LIKE LINE OF lt_registered_hotkeys.