Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(registryclient):[TRI-1470] Improve registry client library #459

Merged
merged 2 commits into from
Jul 31, 2023

Conversation

ds-jkreutzfeld
Copy link

Added new method to retrieve globalAssetIds for a BPN directly. This just uses the existing methods.

Refactored the existing code to be more comprehensible and added some log output. The EndpointDataReference is now renewed on demand if the token expires during the runtime. Renamed DigitalTwinRegistryService#lookupShells to lookupShellIdentifiers to be more precise. The old method points to the new one and is marked as deprecated now.

Added new method to retrieve globalAssetIds for a BPN directly. This just uses the existing methods.

Refactored the existing code to be more comprehensible and added some log output.
The EndpointDataReference is now renewed on demand if the token expires during the runtime.
Renamed DigitalTwinRegistryService#lookupShells to lookupShellIdentifiers to be more precise. The old method points to the new one and is marked as deprecated now.
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

90.5% 90.5% Coverage
0.0% 0.0% Duplication

@@ -24,8 +24,8 @@

/**
* Key object contains required attributes for identify part chain entry node
* @param globalAssetId Id of global asset
* @param bpn bpn of entity
* @param shellId ID of an asset administration shell (note: this is NOT a globalAssetId)

Check notice

Code scanning / CodeQL

Spurious Javadoc @param tags

@param tag "shellId" does not match any actual type parameter of type "DigitalTwinRegistryKey".
* @param globalAssetId Id of global asset
* @param bpn bpn of entity
* @param shellId ID of an asset administration shell (note: this is NOT a globalAssetId)
* @param bpn the business partner number which owns the asset

Check notice

Code scanning / CodeQL

Spurious Javadoc @param tags

@param tag "bpn" does not match any actual type parameter of type "DigitalTwinRegistryKey".
@ds-jkreutzfeld ds-jkreutzfeld merged commit 1c6dd1c into main Jul 31, 2023
@ds-jkreutzfeld ds-jkreutzfeld deleted the feature/TRI-1470 branch July 31, 2023 14:29
ds-jhartmann added a commit that referenced this pull request Mar 1, 2024
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.

3 participants