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.

Sign up for GitHub

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

Jump to bottom

GeoServerRESTReader.existsWmsstore(String,String) has wrong implementation #190

Open
bilzkat opened this issue Apr 27, 2016 · 1 comment
Open

GeoServerRESTReader.existsWmsstore(String,String) has wrong implementation #190

bilzkat opened this issue Apr 27, 2016 · 1 comment
Labels
bug Priority: Medium
Milestone
1.7.1

Comments

@bilzkat
Copy link

bilzkat commented Apr 27, 2016

Line 622 should call the other existsWmsstore() not existsCoveragestore()

Current code:
public boolean existsWmsstore(String workspace, String wsName){
return existsCoveragestore(workspace, wsName, Util.DEFAULT_QUIET_ON_NOT_FOUND);
}

Update:
public boolean existsWmsstore(String workspace, String wsName){
return existsWmsstore(workspace, wsName, Util.DEFAULT_QUIET_ON_NOT_FOUND);
}
@simboss
Copy link
Member

simboss commented Apr 28, 2016

Would you mind providing a PR for this issue?

@simboss simboss added this to the 1.7.1 milestone Apr 28, 2016
bilzkat added a commit to bilzkat/geoserver-manager that referenced this issue Apr 29, 2016
@bilzkat
existsWmsstore defect fix
f728608 
Resolved geosolutions-it#190

existsWmsstore was calling existsCoveragestore instead of overloaded existsWmsstore
@bilzkat bilzkat mentioned this issue Apr 29, 2016
Open
basduineveld pushed a commit to PDOK/DEPRECATED_geoserver-manager that referenced this issue Jul 21, 2016
@bilzkat @basduineveld
existsWmsstore defect fix
bf62fb6 
Resolved geosolutions-it#190

existsWmsstore was calling existsCoveragestore instead of overloaded existsWmsstore
basduineveld pushed a commit to PDOK/DEPRECATED_geoserver-manager that referenced this issue Jul 21, 2016
@bilzkat @basduineveld
existsWmsstore defect fix
1b7cf02 
Resolved geosolutions-it#190

existsWmsstore was calling existsCoveragestore instead of overloaded existsWmsstore
basduineveld pushed a commit to PDOK/DEPRECATED_geoserver-manager that referenced this issue Jul 21, 2016
@bilzkat @basduineveld
existsWmsstore defect fix
213785a 
Resolved geosolutions-it#190

existsWmsstore was calling existsCoveragestore instead of overloaded existsWmsstore
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Priority: Medium
Projects
None yet
Milestone
1.7.1
Development

No branches or pull requests
2 participants
@simboss @bilzkat