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

Fixed #13167 Default location not being set/updated upon check-in #13225

Merged
merged 4 commits into from
Jul 12, 2023

Conversation

inietov
Copy link
Collaborator

@inietov inietov commented Jun 29, 2023

Description

If a location is provided at asset checkin, we use to only set the location_id, but with this PR now we also set the rtd_location_id which is the Default Location, looks like a small change but I kinda struggle to fully understand the logic behind the referenced issue, so I hope it makes sense? I think it does but, not the first time I'm wrong!

I also got rid of an escaping e() method, because the system "control" the locations dropdown so I don't think it's necessary here.

Fixes #13167

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Test Configuration:

  • PHP version: 8.1
  • MySQL version: 8.0.31
  • Webserver version: PHP dev server
  • OS version: Debian 11

@what-the-diff
Copy link

what-the-diff bot commented Jun 29, 2023

PR Summary

  • Modification in AssetCheckinController.php File
    • The log message has been enhanced to now show the new location ID, providing a better understanding of the asset location changes.
    • Adjustments were made to the location_id property of the Asset object. Initially, it was set to an empty string but later updated to reflect the rtd_location_id. This will ensure that the Asset object reflects the correct location information.
    • Comprehensive updates were done to thelocation_id attribute of rtd_location_id and Asset object from the request. This change allows the object properties to be dynamically updated based on the incoming requests.
    • Another modification is the rtd_location_id property of the Asset object which has been updated to the value of the location_id attribute from the request. This ensures that the returned to depot location ID always represents the latest data from the request.

@inietov inietov changed the title Fixes #13167 Default location not being set/updated upon check-in WIP Fixes #13167 Default location not being set/updated upon check-in Jul 6, 2023
@inietov inietov changed the title WIP Fixes #13167 Default location not being set/updated upon check-in Fixes #13167 Default location not being set/updated upon check-in Jul 6, 2023
@inietov inietov marked this pull request as draft July 6, 2023 17:27
@inietov
Copy link
Collaborator Author

inietov commented Jul 6, 2023

Put it as draft because I found an issue, as I put this on the partials view, the option also is shown in the update asset view. As soon as it's resolved I change the draft to pull request again.

@inietov inietov marked this pull request as ready for review July 6, 2023 17:37
@inietov
Copy link
Collaborator Author

inietov commented Jul 6, 2023

Ok, that was fast... I though it would take me more time to remove the inputs from other views.Sorry... Back to ready for review!

@inietov inietov changed the title Fixes #13167 Default location not being set/updated upon check-in Fixed #13167 Default location not being set/updated upon check-in Jul 11, 2023
@snipe snipe merged commit 44231fa into snipe:develop Jul 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants