Commit Graph

594 Commits

Author SHA1 Message Date
Eduardo Ferrari Copat
16258c59b6
Checkout commit: additional user interface (#4120) (#4349)
* Set branch name with commit

* Can't stage when commit is checked out

* Add missing variable

* Remove passing a parameter, since set relies on IS SUPPLIED

* Undo last commit

* Intelligent create branch when commit is checked o

ut

* lint

* Fix cancel buttons (missing commit)

* Code review

* Remove unused variables

* Add extra line

* Remove unused variable

* Add back new branch pop-up. Considers tag.

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-12-28 12:39:26 +01:00
Lars Hvam
704198f521
unit testing test (#4345)
* update deps

* update deps

* remove CHAR01 usages

* update deps

* update deps

* update deps

* all unit tests in syntax now running!

* update deps

* update deps

* update deps

* enable additional exception unit tests

* zcl_abapgit_string_map running

* upd

* login manager running

* update deps, add progress clas

* upd

* update deps

* fix whitespace

* update deps

* upd

* update deps

* update deps

* update deps

* use cl_abap newline

* upd

* run more

* sort

* update deps

* add intf

* update deps

* upd

* add zcl_abapgit_exit

* some xml interfaces

* zcl_abapgit_version running
2020-12-27 18:00:59 +01:00
Lars Hvam
4b2309f952
zif_abapgit_html, remove uses of char1 (#4351) 2020-12-24 08:36:58 +01:00
Marc Bernard
73cba7ff1b
Fix sorting of objects in repo view (#4346)
Regression of #4087. Closes #4339

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-12-22 06:47:45 +01:00
Marc Bernard
f1bf013adf
Refactor: GUI router state transfer (#4336)
Moving page-dependent state changes to separate methods for clarity
2020-12-16 08:23:00 +01:00
Christian Günter
0e4f29efa1
New Repo: Check package existence before showing popup (#4323)
* New Repo: Check package existence before showing p

* fix lint errors

* fix linter

* add more unit tests

* refactor unit tests

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-12-14 14:54:28 +01:00
Jaime Rodriguez Capote
6c87fa3f05
Several repos for same URL (#4313)
* Check URL repo before add

* Unit tests

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-12-12 08:49:38 +01:00
Marc Bernard
d0983cf386
Dialogs: HTML form enhancements (#4264)
* Dialogs: HTML form tweaks

- Minor increase of contrast for background, input- and readonly fields
- Add read-only option for columns of table control

* Add is_empty

* Add init and hidden

* Render

* Remove init

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-12-12 08:45:47 +01:00
Lars Hvam
efaa0e65c0
more generic exception catching (#4307) 2020-12-11 07:46:07 +01:00
Marc Bernard
07cdc0e0e8
Resolve circular reference in /syntax/ (#4292)
Closes #4258

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-12-07 09:13:58 +01:00
Fabian Lupa
ca51d8b0c7
Ensure open in main language opens same abapGit version (#4291)
Fixes #4276

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-12-06 06:36:45 +01:00
Lars Hvam
bf89ac55f6
uninstall: "including subpackages" (#4287) 2020-12-04 08:44:03 +01:00
Lars Hvam
efa6aa9347
Remove two places wih repo class insantiation (#4250)
* zip, method encode_files as public

* refactor instantiation

* remove filter input parameter
2020-11-30 11:23:36 +01:00
Marc Bernard
7f61ebd645
UI: Clear progress bar (#4249)
* UI: Clear progress bar

Clears status/progress bar at end of process and removes leftover messages. 

Closes #3505

* Lint

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-27 10:18:52 +01:00
Marc Bernard
552024ac3f
UI: Improvements for setting pages (#4229)
* UI: Improvements for setting pages

- Better navigation between settings pages
After jumping between setting pages, the back button will go to last page before navigating to the settings (ie "back with bookmark")
- Add commit work

* Lint + router

* Jump repo view to settings

* Lint

* Conflict

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-26 11:54:49 +01:00
Constantinos Sourelis
5d8f4d2033
User interface: "Advanced" menu tooltips (#4214)
* User interface, "Advanced" menu

Added tooltips to the "Remove" and "Uninstall" options in the advanced menu. As a new user, I would revisit the [documentation](https://docs.abapgit.org/guide-online-uninstall.html) to distinguish between the two.

The tooltips are passed as "iv_title" parameters to ZCL_ABAPGIT_HTML_TOOLBAR->add

* Update src/ui/zcl_abapgit_gui_page_repo_view.clas.abap

Adjustments from review

* Update src/ui/zcl_abapgit_gui_page_repo_view.clas.abap

Adjustments from review

* Update zcl_abapgit_gui_page_repo_view.clas.abap

Comply with abaplint line length of 120

* Update zcl_abapgit_gui_page_repo_view.clas.abap

Comply with abaplint's no whitespaces at EOL

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-24 15:43:40 +01:00
Lars Hvam
01d968c1f8
show log, remove title parameter (#4234)
* show log, remove title parameter

the log objec already have the title as part of the object, another input is not required

this aligns the code to always use the title from the object

closes #4223

* zif_abapgit_log, change title to csequence input
2020-11-24 15:41:26 +01:00
Marc Bernard
9efdae315d
Dialogs: Enhance HTML form with table control (#4230)
* Dialogs: Enhance HTML form with table control

- Adds option to maintain data using a table control
- Extends UTs with several cases that were include in #4172
- Table control to be used by "repo settings" #4171

* Border

* Lint

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-24 08:11:53 +01:00
Marc Bernard
aa2f4bb8e7
Resolve hotkey conflict (#4228)
"f" is default for vimium-links. Changed "New Offline" to "o".

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-24 08:04:27 +01:00
Lars Hvam
53790c3770
log, purge() now returns the log (#4222)
* log, purge() now returns the log

For each action, the log should be returned, then it is up to the UI how or if to show or persist it in the session.

First step of #2821

* li_log not needed anymore
2020-11-24 07:58:22 +01:00
Lars Hvam
381d40e35a
ZCL_ABAPGIT_HASH add shorthand methods (#4220)
This adds shorthand methods for the SHA1 calculation of the different git artifacts.
More logic in ZCL_ABAPGIT_HASH, but easier to read and less application logic.
2020-11-23 14:30:31 +01:00
Constantinos Sourelis
e51ca12eb5
Fix uncaught exception of zcl_abapgit_repo_srv (#4215)
When repeatedly navigating "back" to the main screen via the "Repository List" [o] button:
If somewhere in this navigation sequence you Remove a repository,
then continue on via "Repository List",
but later you use the GUI's actual "BACK" button, until you have returned to the main screen where that repository still existed,
a short dump occurs as the exception raised by zcl_abapgit_repo_srv=>get in zcl_abapgit_gui_page_repo_view->render_content propagates unhandled

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-23 08:31:26 +01:00
Marc Bernard
82668de47b
UI: New pages for global and personal settings (#4192)
* Dialogs: New pages for global and personal setting

Redesign of abapGit settings using HTML forms. Two functional changes to old settings page:
- "Development Internal Settings" are only visible when using the developer version (not in standalone program)
- "User-specific hotkey settings" are not available anymore. The default hotkeys continue to work as before.

Global/Personal settings can be reached using hotkeys "x" and "y".

* Lint

* Hide button

* Change to top menu

* Remove old page

* Remove old page

* fix links

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-23 08:24:17 +01:00
Lars Hvam
725d92f98e
bugfix, PR popup sorting (#4206) 2020-11-20 08:51:01 +01:00
Fabian Lupa
123de8fd68
Allow html viewer to be hidden (#4203)
* Allow html viewer to be hidden

for CALL SCREEN in the same session to work

* Move html viewer hiding to router
2020-11-20 08:38:37 +01:00
Marc Bernard
46470da59d
Improvements for uninstall (#4198)
* Improvements for uninstall

- Improve error handling on deletion collecting errors in log
Closes #4197

- Remove limit of 3 loops which allows deleting objects with deeper dependencies
(Follow-up https://github.com/abapGit/abapGit/issues/3228#issuecomment-728800249)

* Exception

* Error msg

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-19 09:31:31 +01:00
Marc Bernard
9e385c264e
UI: Add help buttons to "New Online/Offline" (#4189)
* UI: Add help buttons to "New Online/Offline"

- Add help button to both dialogs linking to docs.abapgit.org
- Clicking the button brings up the documentation inside the GUI HTML viewer
- Enhancement to `zcl_abapgit_gui` and `zcl_abapgit_html_viewer_gui` to properly handle 'Back' when navigating through Internet pages

* Open help in external browser

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-18 07:33:00 +01:00
Fabian Lupa
267f7a0a6e
Fix reset for deletions (#4194)
* Fix reset local not deleting objects

Fixes #4186

* Fix typo

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-17 17:59:01 +01:00
Fabian Lupa
a042442e64
Fix missing td for offline repos (#4193)
Fixes #4190
2020-11-17 10:48:27 +01:00
Marc Bernard
e93cbc5a1f
Move ty_deserialization, ty_step_data, ty_repo_ref_tt (#4181)
* Move ty_deserialization, ty_step_data

Remove dependencies from `zif_abapgit_definitions` (see https://github.com/abapGit/abapGit/issues/4179)

* Move to interface

* Move ty_repo_ref_tt
2020-11-16 08:34:29 +01:00
Marc Bernard
2f31cc6173
UI: Fix HTML validation warnings and error (#4173)
* UI: Fix HTML validation warnings and error

* Back to previous ie=

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-14 16:19:29 +01:00
Lars Hvam
a119c617d3
ALPHA downport, refactor XSTRING_TO_INT (#4177)
* Refactor XSTRING_TO_INT

this will use the built-in type conversion, its less code and most likely also runs faster

* delete METHOD alpha_output

it is only used in one place, and its a simple NUMC that is converted
2020-11-14 16:03:33 +01:00
Marc Bernard
16e8f3d02d
Dialogs: HTML form enhancements (#4172)
* Dialogs: HTML form enhancements

In preparation for https://github.com/abapGit/abapGit/issues/4171, HTML forms were enhanced as follows:

- New integer fields (type "number") with min/max, https://github.com/abapGit/abapGit/issues/3559
- Added support for minlength/maxlength for text fields
- Added support for read-only fields
- Added support for password fields
- New option for adding a help button to the form (with URL link)
- Moved HTML snippets to render method
- Changed parameters to csequence to also accept char variables (like abap_true)
- Minor adjustments to styles

* Lint and fix ul/fieldgroup

* Min/max validation, textarea

* Close textarea tag

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-14 11:28:19 +01:00
Marc Bernard
61599d8225
Dialogs: Change UI texts to title case (#4176)
* Dialogs: Change ui texts to title case

Adjust texts of "New Online" and "New Offline" pages to title case

* Clone
2020-11-14 08:40:50 +01:00
Lars Hvam
c4f75fc5a8
ZCL_ABAPGIT_CONVERT refactor (#4161)
* ZCL_ABAPGIT_CONVERT  refactor

this removes calls to function modules, replacing with class based equivalents, or all custom ABAP implementations

* rename local variables
2020-11-12 09:00:32 +01:00
Lars Hvam
8226aefcfe
add interface ZIF_ABAPGIT_MERGE (#4160)
* move merge types to merge class

this will make the God interface defintions smaller

also reducing INTF -> CLAS references

and making it easier to add an interface for the merge class in the future

* add interface ZIF_ABAPGIT_MERGE
2020-11-12 08:57:34 +01:00
Marc Bernard
b25a67acea
Terminology: Inclusive Language - Part 2 (#4159)
* Terminology: Inclusive Language - Part 2

Change "Master Language" to "Main Language" in UI and comments (documentation and variables next)

* Docs

* Docs

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-11 10:36:38 +01:00
Marc Bernard
b5ba5092bf
Terminology: Inclusive Language - Part 1 (#4158)
Changing "User Master Record" to "User Record"
2020-11-10 15:16:41 +01:00
Lars Hvam
e35df57646
Change TY_SVAL_TT location (#4155) 2020-11-10 14:21:42 +01:00
Lars Hvam
3162b23c39
Move git url repo methods (#4147)
* Move git url repo methods

This moves methods GET_COMMIT_DISPLAY_URL and GET_DEFAULT_COMMIT_DISPLAY_URL from the repo class to new class in new package GIT_PLATFORM.

we want to keep the repo class small, as its quite complex and central

#4085

* fix line length
2020-11-10 06:34:16 +01:00
Marc Bernard
5df956a37a
Toggle display of hidden characters on diff page (#4136)
* Toggle display of hidden characters on diff page

Time for something new...

I added a toggle switch to the diff page that allows you to show otherwise hidden characters.
- CR end-of-line 
- Spaces
- Tabs
- Byte-order-mark

Closes https://github.com/abapGit/abapGit/issues/2124
Closes https://github.com/abapGit/abapGit/issues/3676
Improves https://github.com/abapGit/abapGit/issues/2804

* Add ini

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-07 09:54:38 +01:00
Lars Hvam
85404ce72f
Refactor ZCL_ABAPGIT_PERSIST_BACKGROUND (#4123)
+ remove MT_JOBS
+ add GET_BY_KEY that can be used instead of LIST

closes #1455
2020-11-07 09:52:00 +01:00
Marc Bernard
5c1298697d
Trigger main command when pressing enter in form (#4102)
* Triggers main command when pressing enter in form

Adds hidden button that triggers main command when pressing the enter key

* Lint

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-05 09:46:21 +01:00
Marc Bernard
b408cad115
Backup and restore for abapGit persistency (#4137)
* Backup and restore for abapGit persistency

Backup complete content of abapGit persistency to a ZIP file and restore when necessary.

* Messages

* 702

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-05 09:28:48 +01:00
Marc Bernard
87a3806ce1
UI: Unify rendering of transport, user names - part 2 (#4133)
- Added icon for user
- Changed icon for transport to truck
- Adapted "staging" page 
- Renamed CSS-classes because of conflict with classes used on staging page

Screenshots #4097
Closes #4098

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-04 14:51:34 +01:00
Frederik Hudák
990441dbe0
Retrieve transport and changed by for deletions (#4056)
* Retrieve transport and changed by for deletions

* Refactor

* Remove breakpoint

* Update src/ui/zcl_abapgit_gui_page_stage.clas.abap

Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>

* Update src/ui/zcl_abapgit_gui_page_stage.clas.abap

Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>

* Update src/ui/zcl_abapgit_gui_page_stage.clas.abap

Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>

* Abaplint

* ABAPLINT

Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-03 08:52:49 +01:00
Lars Hvam
c2492dd47e
delete unused methods (#4125)
* ZCL_ABAPGIT_REPO_FILTER

delete ZCL_ABAPGIT_REPO_FILTER->GET_PACKAGE
ZCL_ABAPGIT_REPO_FILTER delete MV_PACKAGE

* testclass, delete method given_fstate

* delete ZCL_ABAPGIT_GUI_PAGE_PATCH->IS_EVERY_CHANGE

delete method delete ZCL_ABAPGIT_GUI_PAGE_PATCH->IS_EVERY_CHANGED_LINE_PATCHED

* delete ZCL_ABAPGIT_OBJECTS_SUPER->CHECK_TIMESTAMP
2020-11-03 08:45:21 +01:00
Lars Hvam
df652f0436
delete various unused local variables (#4126) 2020-11-03 08:43:04 +01:00
Marc Bernard
6779e90448
UI: Sort "Switch origin" by PR number (#4127)
- Sort descending by PR number
- Rename menu items

Closes https://github.com/abapGit/abapGit/issues/4031

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-03 08:41:06 +01:00
Marc Bernard
e6acd28bda
Fix invalid HTML (#4128)
* Fix invalid HTML

- Fix HTML in case changelog contains `<, >, &` characters

* Invalid unquoted attribute value

* No tags after </body>
2020-11-03 08:38:02 +01:00