LocationPickerActivity.java: rewrite method scope, name, and documentation

Before this commit, the method name was unclear and the documentation did not fully explain the method's behavior. Additionally, it was a public method.

This commit renames the method (also changing calls to it), adds more documentation, and changes the method scope from public to private.
This commit is contained in:
Jason Whitmore 2024-07-01 16:13:26 -07:00
parent a8bdbbc62d
commit d941c2fa8e

View file

@ -331,7 +331,7 @@ public class LocationPickerActivity extends BaseActivity implements
moveMapToMediaLocation();
modifyLocationButton.setOnClickListener(v -> onClickModifyLocation());
removeLocationButton.setOnClickListener(v -> onClickRemoveLocation());
showInMapButton.setOnClickListener(v -> showInMap());
showInMapButton.setOnClickListener(v -> showInMapApp());
darkThemeSetup();
}
@ -388,8 +388,9 @@ public class LocationPickerActivity extends BaseActivity implements
/**
* Show the location in map app. Map will center on EXIF location, if available.
* If there is no EXIF data, the map will center on the commons app map center.
*/
public void showInMap() {
private void showInMapApp() {
//Check to see if EXIF location data is available
if(cameraPosition != null){
Utils.handleGeoCoordinates(this,