Commit Graph

2899 Commits

Author SHA1 Message Date
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
Fabian Lupa
465c88fd0f
Adjust password field visible length (#4138) 2020-11-05 08:42: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
Fabian Lupa
71fe8683b7
Workaround for special chars in password dialog (#4135)
at the cost of limiting password length to 255 characters

Fixes #4055
2020-11-04 14:24:45 +01:00
Lars Hvam
a53776ab5a
delete unused method in ZCL_ABAPGIT_OBJECT_CHDO (#4130)
* delete unused method

delete unused method GET_GENERIC in class zcl_abapgit_object_chdo

* enable rule unused_methods
2020-11-03 14:55:01 +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
Mike Pokraka
c1b2cf10a0
Update unit tests to use CI (#4106)
* Update unit tests to use CI

* Make test repo configurable

* Remove local tests and use CI

* Lint fixes

* Replace local tests with CI

* Add Type Group constants to abaplint

* Update constants and replace assert_true

* Revert constants

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-03 08:49:22 +01:00
Lars Hvam
266ffb2a7c
/git/: remove unused param, add unit test (#4124)
UPLOAD_PACK_BY_BRANCH
- remove unused parameter EO_BRANCH_LIST

add unit test to ZCL_ABAPGIT_GIT_UTILS
2020-11-03 08:47:12 +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
Lars Hvam
307177bcbf
1.101.0 (#4122)
* 1.101.0

* v1.101.0
2020-11-02 15:03:13 +01:00
Marc Bernard
f6cd2e12bd
Fix DDIC activation issues (#4111)
* Fix DDIC activation issues

- Avoid double activation of DDIC objects in case of "LATE" processing
- Derive DDIC id for XINX and MCID (same as for INDX)

Closes #4103

* Lint

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-02 09:40:33 +01:00
Marc Bernard
29aea372a5
UI: Adjustments to Repo Palette (#4079)
* UI: Adjustments to Repo Palette

- Sort alphabetically
- Pad offline icon (using CSS)

* CSS

* CSS

* Remove css changes

Css is fixed in #4112

Co-authored-by: Lars Hvam <larshp@hotmail.com>
Co-authored-by: Alexander Tsybulsky <sbcgua@users.noreply.github.com>
2020-11-02 09:38:33 +01:00
Marc Bernard
e7b5281dde
Remove todo about unknown user id (#4113)
It's not feasible to use transports to determine who changed an object last (see https://github.com/abapGit/abapGit/pull/4056#issuecomment-719956576).

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-02 09:33:30 +01:00
Marc Bernard
20c1dc0bda
Remove obsolete late_deser from metadata (#4116)
The field `late_deser` in `ZIF_ABAPGIT_DEFINITIONS=>ty_metadata` is obsolete:

```abap
  TYPES:
    BEGIN OF ty_metadata,
      class        TYPE string,
      version      TYPE string,
      late_deser   TYPE abap_bool, " refactor: can be removed later. replaced by steps
      delete_tadir TYPE abap_bool,
      ddic         TYPE abap_bool,
    END OF ty_metadata .
```

The PR removes the remaining usage with one exception: `ZCL_ABAPGIT_OBJECTS_BRIDGE->GET_DESERIALIZE_STEPS` still depends on the field. In order to remove it completely, the plug-in interface `zif_abapgitp_plugin` (https://github.com/abapGit/abapGit-Plugins/blob/master/src/zif_abapgitp_plugin.intf.abap) will have to be changed.

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-02 09:30:55 +01:00
Marc Bernard
dd37881589
702: Fix syntax error and missing check variant (#4117)
* 702: Downport 

No chaining with CHANGING

* 702: Downport

* Syntax check variant

* Too many fields warning

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-02 09:29:09 +01:00
Marc Bernard
bdc44f7476
Remove self-references (#4115)
* Remove self-references

Remove unnecessary self-references (me->)

https://github.com/SAP/styleguides/blob/master/clean-abap/CleanABAP.md#omit-the-self-reference-me-when-calling-an-instance-method

* 702: Fix syntax error

No chaining with EXPORTING & CHANGING

* remove addiontal me, update abaplint config

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-02 09:25:27 +01:00
Alexander Tsybulsky
614a8d60a3
new icons and css adjustments (#4112)
Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-02 09:16:59 +01:00
mariusraht2
ec99d16ec0
Checkout commit: Add menu entry and finish implementation (#4101)
* Implementation

* Add menu entry for checkout commit

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-30 10:24:27 +01:00
Marc Bernard
666dfbda66
UI: Unify rendering of branch, package, transport, user names (#4097)
* UI: Unify rendering of branch, package, user names

Branch name was already rendered "boxed with a leading icon". The same is now done for package and user names.

See `zcl_abapgit_gui_chunk_lib->render_branch_name, render_package_name, render_user_name`.

Closes #4059

* Lint %26 transport

* Unknown user

* Transport css

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-30 10:13:09 +01:00
Marc Bernard
5d480d8e8d
Fix jump links on diff page (#4096)
Some files that existed locally didn't have a link. After #4084, the state can be evaluated properly to determine if local object exists.

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-30 10:01:42 +01:00
Mike Pokraka
9cc515340a
Add option to specify own repo path (#4107)
Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-30 09:58:34 +01:00
Marc Bernard
92ba04d00e
TABL: Fix "exists" check (#4105)
Tables could exist in "new" (or other) version that is not included in nametab.
2020-10-30 09:53:54 +01:00
Philipp R
27aca4b382
Remove System Id from SOTR object type (#4104)
Co-authored-by: Rembold Rembold <philipp.rebold@sap.com>
2020-10-29 18:58:54 +01:00
mariusraht2
b75c6f8515
Add selected commit to repo persistence (#4095)
* Implementation

* Remove old source code

* Clear selected_commit on switch_repo_type

* Reset selected commit or branch on select

* Don't clear the branch name if commit selected
2020-10-29 06:48:41 +01:00
Lars Hvam
1c63b6b2d7
bugfix, the selected commit is not the current commit (#4091)
* bugfix, the selected commit is not the current com

* update
2020-10-28 13:54:33 +01:00
Frederik Hudák
888bbcb27e
Disable sorting for action column (#4087)
* Disable sorting for action column

* abaplint

* Revert renamed methods
2020-10-28 12:23:00 +01:00
mariusraht2
6b336c58f0
ZCL_ABAPGIT_REPO_ONLINE: Change methods fetch_remote and push to be able to retrieve and push objects also by commit hash (#4077)
* Implementation

* Add changes to push method

* Reset commit hash if branch is being switched

* abapLint: Remove whitespace

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

Co-authored-by: Lars Hvam <larshp@hotmail.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
2020-10-28 11:56:56 +01:00
mariusraht2
a4172c6acc
ZCL_ABAPGIT_SERVICES_GIT: Add new method checkout_commit to let the user check out commits (#4076)
* Implementation

* abapLint: Omit exporting

* Omit obsolete variables

* Add caller for checkout_commit

* Extend hash column for 1 character

* Add reset_remote to set_sha1

* Merge branch 'master' into add_checkout_commit

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-28 11:08:39 +01:00
mariusraht2
69b6c3c38a
Implementation (#4086) 2020-10-28 10:41:22 +01:00
Domi Bigl
15dcf41984
add user exit for commit display URL (#4073)
* add user exit for commit display URL

  METHODS adjust_display_commit_url
    IMPORTING iv_repo_url    TYPE zif_abapgit_persistence=>ty_repo-url
              iv_commit_hash TYPE zif_abapgit_definitions=>ty_sha1
    CHANGING  cv_display_url TYPE zif_abapgit_persistence=>ty_repo-url

* adjust UT

* provide also repo name and key in the exit

* Update ref-exits.md

* Update ref-exits.md

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-28 07:00:37 +01:00
Marc Bernard
ba0c1257c8
Fix state calculation for deleted objects (#4084)
* Fix state calculation for deleted objects

In case objects are deleted locally or remotely, abapGit now compares against the last known state (checksum). 

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

* Added test case

Covers known local and remote states

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-28 06:57:27 +01:00
Marc Bernard
f699812b02
UI: Adjustments to Command Palette (#4080)
- Add prefix for utilities and help commands
- Remove leading spaces

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-28 06:46:53 +01:00
Christian Günter
d14393dac5
TABL: Skip data check for GTTs (#4075)
* TABL: Skip data check for GTT

fixes #4074

* undo case change

* TABL: use fm DD_EXISTS_DATA for data check

* remove formatting change

* Data existence check only for DB tables

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-28 06:45:04 +01:00
Marc Bernard
46f504a0e8
UI: Adjustments to Hotkeys (#4083)
- Add hotkey "f" for "new offline"
- Add hotkey "c" for "syntax check"
- Fix #3927
2020-10-28 06:42:40 +01:00
mariusraht2
3ec07c6493
ZCL_ABAPGIT_SERVICES_GIT: Change signature of method checkout_commit_build_popup (#4072)
* Implementation

* Remove unrelated pretty printer changes

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-27 16:14:16 +01:00
mariusraht2
5aebc0c88e
ZCL_ABAPGIT_SERVICES_GIT: Add method checkout_commit_build_list to build the popup selection table for commits (#4070)
* Implementation

* abapLint: Omit exporting and avoid void types

* Clear all exporting parameters first

* Remove unrelated pretty printer changes
2020-10-27 16:05:04 +01:00
mariusraht2
761bbe51af
ZCL_ABAPGIT_GIT_COMMIT: get_by_branch should return parent commit hash too (#4069)
* Implementation

* Turn exporting to returning parameter

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-27 10:01:15 +01:00
Marc Bernard
caea4865bf
Integration of unit test with CI (#3993)
Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-27 08:15:52 +01:00
mariusraht2
6f01b691a8
Implementation (#4066)
Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-27 08:11:13 +01:00
mariusraht2
9efa0c916a
Implementation (#4067) 2020-10-27 08:04:51 +01:00
Jaime Rodriguez Capote
d6fbcc1fb6
Fix repo view class name importing ZIP (#4065) 2020-10-26 10:39:16 +01:00
Fabian Lupa
ceeb95eec1
Fix arrow direction for sort order (#4062) 2020-10-26 08:06:28 +01:00
Christian Günter
2d0729bc98
Exit logic should only be applied for main sel scr (#4060)
fixes #4058
2020-10-24 08:14:23 +02:00
Frederik Hudák
d04b1b057c
DEVC - Clear obsolete fields (#4054)
* DEVC - Clear obsolete fields

* Remove on deserialize side
2020-10-22 19:04:41 +02:00
Marc Bernard
efa0935a5c
Set validity of default transport to one day (#4051)
Closes #4050
2020-10-22 08:43:02 +02:00
g-back
42f52990e5
ABAP Syntax Highligher: Re-Add New Keywords (#4047)
* add minimal unit test

* readd new keywords to abap syntax highlighter


- removed duplicates from keywords
- removed deprecated pseudo comment
- explicitly remove duplicates from keywords to avoid needless dumps

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-19 15:20:17 +02:00
mariusraht2
7ffb4ff91d
Move commit-specific methods from ZCL_ABAPGIT_BRANCH_OVERVIEW to new class ZCL_ABAPGIT_GIT_COMMIT (#4026)
* Implementation

- Create new class ZCL_ABAPGIT_GIT_COMMIT
- Move commit-specific methods from ZCL_ABAPGIT_BRANCH_OVERVIEW to ZCL_ABAPGIT_GIT_COMMIT
- Change methods visibility (from private to public) and type (from instance to class method)

* Remove ##NO_TEXT

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

* Rename get to get_from_remote

* abapLint: Remove space

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

* abapLint

- Remove space
- Remove unused type in ZCL_ABAPGIT_BRANCH_OVERVIEW

* abapLint: Remove space

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

* Remove obsolete class attribute

* Separate get_by_branch and get_by_commit

* Remove duplicate char

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

* Remove iv_repo_name; replace its usage by url

Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-19 11:24:38 +02:00
mariusraht2
54ce6771bd
Issue #4027: Resolve possible crash of AG and add option to be able to suppress showing commit hash on Code Inspector result page (#4028)
* Replace LEAVE with CALL SCREEN

* Add new option iv_show_commit

Add new option iv_show_commit to GUI_CHUNK_LIB to be able to suppress showing commit hash on GUI_PAGE_CODE_INSP page

* abapLint: Replace 'EQ' by '='

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

* Hide commit hash on other pages too

Hide commit hash on ZCL_ABAPGIT_GUI_PAGE_REPO_SETT and ZCL_ABAPGIT_GUI_PAGE_SYNTAX pages

Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-19 11:19:02 +02:00
Lars Hvam
356d382723
rollback syntax highlighter changes (#4045)
* rollback syntax highlighter changes

it contains duplicates, resulting in dump

closes #4044

* Update zcl_abapgit_syntax_abap.clas.abap
2020-10-19 09:46:58 +02:00
Marc Bernard
bd46ab764c
Update of ABAP syntax highlighter (#4042)
Some clean-up and additions based on https://github.com/microsoft/monaco-languages/pull/112

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-18 09:10:35 +02:00
mariusraht2
a366e024ea
full_tree: Rename iv_branch to iv_parent (#4041)
Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-18 09:04:55 +02:00
Fabian Lupa
5504c67669
Include inactive objects in show changes only (#4040)
Fixes #4022
2020-10-18 08:58:08 +02:00
Lars Hvam
ff962f82e8
remove INTERFACE LOAD statement (#4035) 2020-10-17 11:07:28 +02:00
Fabian Lupa
60f205d391
Remove remaining 2FA objects (#4039)
Fixes #4037

See #3839 #3150
2020-10-17 11:04:56 +02:00
Marc Bernard
db22013699
702: Fix ZCL_ABAPGIT_PERSISTENCE_USER->read_repo_config (#4038)
Closes #4036
2020-10-16 17:59:25 +02:00
mariusraht2
53adc30708
Add pull_by_commit (#4023)
* Implementation

* Clean-up

* Remove iv_branch

* abapLint

* Remove optional for iv_commit_hash

* Change method parameters

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-16 12:30:43 +02:00
Marc Bernard
e6fb4e45a4
Refactor user persistence (#4003)
* Refactor user persistence

Instead of selecting the settings from the DB for each "get", user settings are now kept in instance memory. Simplifies code, too.

* Lint

* Replace commit work

* Change risk level

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-16 09:18:01 +02:00
mariusraht2
9ac7630711
ZABAPGIT_FORMS: Remove obsolete form package_popup and related callback handler (#4029)
* ZABAPGIT_FORMS: Remove obsolete methods

- Remove branch_popup
- Remove package_popup

* Readd branch_popup

* Readd branch_popup

* Remove related callback handler

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-16 09:11:18 +02:00
Lars Hvam
958c894d4b
issue #4032, downport (#4033) 2020-10-16 08:36:23 +02:00
Marc Bernard
4ebde6d1a3
CLAS: Avoid unnecessary serialize of testclasses include (#4013)
* CLAS: Avoid unnecessary serialize of testclass file

Unit tests are only functional if the corresponding flag is set (`seoclassdf-with_unit_tests`). Therefore, if the flag is not set, there's no need to serialize this include.

* Added version to select

* Switch to seo function

* Set  mv_skip_testclass

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-14 09:38:29 +02:00
Marc Bernard
5068e7553c
CLAS: Performance APACK (#4016)
- Replace `select count` with `select single`
- Remove unnecessary loops
- Keep SAP apack manifest during deserialize (if available)

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-13 16:49:21 +02:00
Marc Bernard
1c8c216a5c
Simplify Ignore Logic (#3996)
* Simplify Ignore Logic

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

* Update docs regaring ignore logic

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-13 16:27:28 +02:00
Lars Hvam
f6ad8c668d
XSLT, fix syntax error (#4019) 2020-10-13 16:06:55 +02:00
mariusraht2
90b0f9fae5
upload_pack_by_commit: Be able to get remote objects by any commit hash (#3836)
* Create pull.yml

* Update zcl_abapgit_git_transport.clas.abap

* Update zcl_abapgit_git_transport.clas.abap

* Update pull.yml

* Update pull.yml

* Update zcl_abapgit_git_transport.clas.abap

* Delete pull.yml

* Update zcl_abapgit_git_transport.clas.abap

* Rename upload_pack to upload_pack_by_branch

* Renamed missed references

* Introduce new method upload_pack_by_commit

* abapLint: Removed forbidden pragmas/comments

Forbidden pseudo comment
Forbidden pragma

* Replaced numc2 by n length 2

* Removed unneccessary changes of pretty printer

* Removed restriction of deepen_level

* N alone is of length 1 and not numeric

* abapLint

* Remove branch name

* Merge redundant source code into upload_pack

* Make upload_pack private

* abapLint

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

* Update src/git/zcl_abapgit_git_transport.clas.abap

Co-authored-by: Lars Hvam <larshp@hotmail.com>

* Omit receiving

* Omit exporting

Co-authored-by: mariusraht1 <mariusraht+1@gmail.com>
Co-authored-by: pull[bot] <39814207+pull[bot]@users.noreply.github.com>
Co-authored-by: Lars Hvam <larshp@hotmail.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
2020-10-13 15:58:39 +02:00
Marc Bernard
b2790bc1a2
XSLT: Fix activation sequence (#4017)
* XSLT: Fix activation sequence

Save first, then activate (and no need for activation queue)

* Add error handling
2020-10-13 10:43:23 +02:00
Alexander Tsybulsky
19c49fd960
Bugfixes in html_form and Finetune add pages to new event (#4011)
* refactor validation in html_form + UT

* minor UT tweak

* subtle form bugfix (boolc return space string)

* cleanup add forms

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-12 16:40:24 +02:00
Alexander Tsybulsky
4b6c424465
html event abstraction, phase 5 - case insensitive query and form_data (#4007)
* case-insensitive string map

* refactor gui_event and its UTs

* remove query uppercased in pages

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-11 10:47:37 +02:00
Lars Hvam
45bc9ed91d
update linter configuration (#4008) 2020-10-11 10:40:18 +02:00
Marc Bernard
c27a0c0b38
Update ZCL_ABAPGIT_USER_MASTER_RECORD (#3994)
* Update ZCL_ABAPGIT_USER_MASTER_RECORD

- Issue: If the user does not exist in the system, the BAPI will fail but still the user still will be added to the gt_user.
- Fixed the issue, if user is invalid it wont be added to the list.
- Refactored the class and added check_user_exist method.
- Added unit tests.

* Sy-uname

* Lint

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-10 07:52:57 +02:00
Marc Bernard
5a0acf7a5a
SHMA: Auto generate (#4005)
Automatically generate area class after deserializing. Closes https://github.com/abapGit/abapGit/issues/1201

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-09 13:13:22 +02:00
Marc Bernard
c0192df9ec
Close some minor ToDos (#3995)
* Close some todos 

FUGR does not have any class includes

* Remove huh?

* Move WDYN activation to serializer

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-09 13:11:24 +02:00
Marc Bernard
784975fefc
Update object serializer descriptions (#4006)
Helps to identify the object type of a serializer class

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-09 12:58:31 +02:00
Marc Bernard
0adb9dec58
Add Offline: Uppercase package name (#4002) 2020-10-09 12:30:07 +02:00
Marc Bernard
b7b5c2e62e
Change ESCAPE key to work like BACK (#3989)
Pressing ESC used to exit abapGit completely. Changed it to work like F3 (BACK), which makes more sense especially in HTML form. You can still exit from anywhere using SHIFT+F3.

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-07 09:14:10 +02:00
Lars Hvam
f9d064829a
remove more ALIASes (#3986)
most of these are not in use, but was also public
2020-10-07 09:05:28 +02:00
Lars Hvam
7339e0ec47
delete ltcl_dangerous (#3984) 2020-10-07 09:02:36 +02:00
Lars Hvam
4e96acd4be
use HEAD symref as default branch (#3983)
* use HEAD symref as default branch

closes #3974

* autodetect
2020-10-07 08:59:30 +02:00
Marc Bernard
9672f5b030
Adjust name prefix for types (#3991)
* Part 1

* Part 2

* Part 3

* Lint

* Part 5

* Part 6

* Part 7

* Part 8

* Part 9

* Enforce type naming prefix

All types need to start with TY_...

* Fix local types
2020-10-07 08:54:15 +02:00
g-back
419bbfc24d
ZCL_ABAPGIT_HASH: Change FM call to method call (#3980)
* change function module call to method call

* add test case for valid XSTRING value

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-06 08:40:48 +02:00
Alexander Tsybulsky
7c0ec067a1
rename repo related pages (#3982) 2020-10-06 08:22:22 +02:00
Marc Bernard
19916a5909
Remove reference to helper (#3981) 2020-10-05 17:34:57 +02:00
Marc Bernard
42c909e688
Refactor: Remove dynamic instantiation of ZCL_ABAPGIT_MESSAGE_HELPER (#3966)
* Refactor: Remove dynamic instantiation of ZCL_ABAPGIT_MESSAGE_HELPER

Consolidate `ZCL_ABAPGIT_MESSAGE_HELPER` features into `ZCX_ABAPGIT_EXCEPTION` and removes helper.

* Remove helper

* Pretty

* Update chunk_lib

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-05 16:50:36 +02:00
Harry Abrahams
6360ccd08a
Fix for SFPF Clear CACHE_INFO (#3954)
* Clear CACHE_INFO

#3868

* Indentation corrections

* Add Spaces for Parameters

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-05 16:44:28 +02:00
Alexander Tsybulsky
6941d9824a
html event abstraction, phase 4 - form_data (#3975)
* event->post_data logic

* part of pages, commit test

this is also long text
to test

* string_map: port strict() feature

* pages refactoring part 2

* linter

* docs

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-05 09:33:01 +02:00
Lars Hvam
c573b94a57
ZCL_ABAPGIT_CONVERT additional unit tests (#3976) 2020-10-05 09:30:43 +02:00
Marc Bernard
25774396e6
Dialogs: add offline repo HTML form (#3960)
* Dialogs: add offline repo HTML form

* Dialogs: add offline repo HTML form

* Dialogs: add offline repo HTML form

* Lint

* Lint

* Chaining

* Set required attribute

* Check for empty display name

* Change validation

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-03 10:18:45 +02:00
Marc Bernard
24436879b6
Improvements for uninstall (#3959)
* Improvements for uninstall

Uninstall now catches exceptions and tries up to 3 times to remove an object. Exceptions are only raised if after 3 tries some objects still remain. 

Tested with https://github.com/larshp/abapPGP

Closes https://github.com/abapGit/abapGit/issues/379 and https://github.com/abapGit/abapGit/issues/3949, and probably #3228

* Add SHLP

also ordered when by priority

* Conflict

* Move counter

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-03 09:17:11 +02:00
Marc Bernard
9704ce6708
Fix ZIF_ABAPGIT_OBJECT~EXISTS methods (#3967)
* Fix ZIF_ABAPGIT_OBJECT~EXISTS methods

* Fix EXISTS

* CMPT

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-10-02 10:23:00 +02:00
Lars Hvam
37ae397853
1.100.0 (#3970)
* 1.100.0

* v1.100.0
2020-10-01 11:08:01 +02:00
Marc Bernard
ef234a20fb
New object type CMOD - Customer Enhancement Project (#3947)
* New object type CMOD - Customer Enhancement Project

Test repo https://github.com/abapGit-tests/CMOD

* CMOD

* boolc

Co-authored-by: Lars Hvam <larshp@hotmail.com>

* Update src/objects/zcl_abapgit_object_cmod.clas.abap

Co-authored-by: g-back <27279305+g-back@users.noreply.github.com>

Co-authored-by: Lars Hvam <larshp@hotmail.com>
Co-authored-by: g-back <27279305+g-back@users.noreply.github.com>
2020-10-01 09:49:48 +02:00
Marc Bernard
1e93c1d3d3
Fix syntax errors in 702 (#3962)
Only relevant if trying to run dev version in 702
2020-09-29 19:28:42 +02:00
Marc Bernard
c9da76b3ee
Fix error from CL_PACKAGE_FACTORY=>LOAD_PACKAGE (#3958)
* Fix error from CL_PACKAGE_FACTORY=>LOAD_PACKAGE

* Variable

* Change to T100 messages

* super->constructor( )

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-09-29 17:29:02 +02:00
pokrakam
00e01c7232
Extend unit tests to handle edge cases (#3952)
Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-09-29 07:46:25 +02:00
pokrakam
85fe733171
Object type PDTS - Workflow Task (#3711)
* Adapt trial code from christianguenter2

* Add changed_by

* Adapt trial code from christianguenter2

* Add changed_by

* Add unit test mock class

* Update unit tests

* Unit tests %26 lint fixes

* De-Linting

* More lint fixes

* More lint experimenting

* More de-linting

* Update unit tests

* Turnaround test (WIP)

* More XML turnaround testing

* Turnaround test now working

* Clean up code

* Enable experimental mode

If we want to merge into master, it will only run for daredevil experimentalists

* Update src/objects/zcl_abapgit_object_pdts.clas.testclasses.abap

Co-authored-by: Lars Hvam <larshp@hotmail.com>

* Update src/objects/zcl_abapgit_object_pdts.clas.abap

Co-authored-by: Lars Hvam <larshp@hotmail.com>

* 702 compatibility

* More 702 compatibility

* Removed container texts and system elements

Temporary(?) measure as they are being too troublesome, changing XML on successive serializations and suchlike.

* Caught exception

* General refactoring

* Remove NOTEXT pragmas and directives

* De-linting

* Remove diff in transport class

* Refactoring WIP

* Refactoring - WIP

* Refactor serialization

* De-lint

* Update container serialization

* Refactor deserialization WIP

* Object type PDTS - Workflow Task (experimental)

* Object PDTS Alpha version

* De-lint

* Correct CI dependency issues

* Check for critical unit test setting

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-09-29 07:41:35 +02:00
Marc Bernard
404a08fdb2
New object type WDCA - Web Dynpro Application Configuration (#3941)
* WDCA

* WDCA

* Dependencies

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-09-29 07:35:32 +02:00
Marc Bernard
5bde9bd549
Fix syntax errors in 702 (#3955)
* Fix syntax errors in 702

* Fix syntax errors in 702
2020-09-28 19:58:31 +02:00