Skip to content

Add Business Processess if they are not in source control#686

Merged
isc-tleavitt merged 5 commits intomainfrom
issue-676
Jan 14, 2025
Merged

Add Business Processess if they are not in source control#686
isc-tleavitt merged 5 commits intomainfrom
issue-676

Conversation

@isc-etamarch
Copy link
Contributor

Fixes #676

@isc-etamarch
Copy link
Contributor Author

isc-etamarch commented Jan 14, 2025

There is no 'new document' hook when creating a new business process (it just registers as opening existing document), so we have to work around this.

@isc-pbarton This is only a partial solution. This works only for compiled business processes, since there is no easy way to check (that I know of) if a class is a business process before it has been compiled. Though once a business process is compiled, it is reloaded, thus causing it to be added to source control.

@codecov-commenter
Copy link

codecov-commenter commented Jan 14, 2025

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 62.50000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 40.50%. Comparing base (730e6fe) to head (506e942).

Files with missing lines Patch % Lines
cls/SourceControl/Git/Extension.cls 62.50% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #686      +/-   ##
==========================================
- Coverage   40.54%   40.50%   -0.04%     
==========================================
  Files          23       23              
  Lines        3137     3145       +8     
==========================================
+ Hits         1272     1274       +2     
- Misses       1865     1871       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@isc-pbarton
Copy link
Collaborator

isc-pbarton commented Jan 14, 2025

This looks good to me. Can we make the same fix for Business Rules as well? And perhaps add a code comment that we are making up for an upstream issue.

@isc-etamarch
Copy link
Contributor Author

This looks good to me. Can we make the same fix for Business Rules as well? And perhaps add a code comment that we are making up for an upstream issue.

Does this issue occur for Business Rules? From my testing it seems like they get added automatically when created

@isc-pbarton
Copy link
Collaborator

I'm seeing it with business rules if I start from the production configuration page. Add a new process of type EnsLib.MsgRouter.RoutingEngine, check "Auto-create rule", then open the auto-created rule. It has the "Add to Source Control" option on it. This is on IRIS 2023.1.3

@isc-etamarch
Copy link
Contributor Author

@isc-pbarton That should work now too, though the Rule Editor does not call hooks when opened, so this will only work on Save/Compile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Business processes are not automatically added to source control

4 participants