Ticket #6490 (closed: fixed)
Catalog menu/features are unusable if facility is not set to ISIS
Reported by: | Russell Taylor | Owned by: | Russell Taylor |
---|---|---|---|
Priority: | major | Milestone: | Release 2.4 |
Component: | MantidPlot | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | #6512 | Tester: | Peter Peterson |
Description
I don't think this is new, but if your facility not ISIS none of this can be used as it's only set up for ICAT3. No matter what you put in the login box, you will get back an error without explanation. Surely we should do better in case someone does try to use it. Short-term possibilities might be one or more of the following:
- Remove the menu item entirely (or grey it out) if the facility is not set to SNS.
- Insert a check that pops up a dialog box explaining that this only works for ISIS.
- Add a statement to CatalogLogin's setOptionalMessage that this only works for ISIS.
- Add a property to CatalogLogin called 'Facility' (which will only have the option of ISIS for now).
Attachments
Change History
comment:1 Changed 8 years ago by Nick Draper
- Owner set to Owen Arnold
- Status changed from new to assigned
comment:3 Changed 8 years ago by Peter Peterson
I spoke with Shelly about this. Apparently we had an ICAT3 server for a couple of months a while ago. We no longer do. The SOAP endpoint listed in the Facilities.xml is an ICAT4 endpoint and ICAT3/4 have radically different apis.
comment:4 Changed 8 years ago by Owen Arnold
refs #6490. Disable ICAT on facilities.
Also improve log message on the login algorithm.
Changeset: 7f669eab388ee6e9b3508a59889924454bcdcd98
Changed 8 years ago by Owen Arnold
- Attachment Screen Shot 2013-01-31 at 09.06.26.png added
New error message
comment:6 Changed 8 years ago by Owen Arnold
- Status changed from accepted to verify
- Resolution set to fixed
Tester:
- Change your facility to SNS
- Go to Catalog -> Login
- Enter a username + password (doen't matter what)
- Should get an error message similar to that in the attached screenshot.
- Repeat for HFIR and ILL, should get a similar error message to the screenshot.
I've also opened up #6512 off the back of this ticket.
comment:7 Changed 8 years ago by Roman Tolchenov
- Status changed from verify to verifying
- Tester set to Roman Tolchenov
comment:8 Changed 8 years ago by Roman Tolchenov
- Status changed from verifying to closed
Confirm the error message for SNS, HFIR and ILL
comment:9 Changed 8 years ago by Russell Taylor
- Status changed from closed to reopened
- Resolution fixed deleted
The <archive> tag relates to the archive search in the FileFinder, so should not have been removed for the SNS.
comment:10 Changed 8 years ago by Russell Taylor
Re #6490. Put back SNS <archive> tag.
This is required for the archive search within the FileFinder. Although it uses ICAT underneath, it is unrelated to the Catalog menu.
Changeset: 9d37d0a719bb7dc759f50cf879261e9db7d21f70
comment:11 Changed 8 years ago by Russell Taylor
- Owner changed from Owen Arnold to Russell Taylor
- Status changed from reopened to accepted
comment:12 Changed 8 years ago by Russell Taylor
- Status changed from accepted to verify
- Resolution set to fixed
Should be retested by someone at the SNS with archive access, making sure that it works.
comment:13 Changed 8 years ago by Peter Peterson
- Status changed from verify to verifying
- Tester changed from Roman Tolchenov to Peter Peterson
comment:14 Changed 8 years ago by Peter Peterson
- Status changed from verifying to closed
I rechecked the archive search for a TOPAZ and POWGEN file. It worked.
comment:15 Changed 8 years ago by Owen Arnold
refs #6490. Disable ICAT on facilities.
Also improve log message on the login algorithm.
Changeset: 7f669eab388ee6e9b3508a59889924454bcdcd98
comment:16 Changed 8 years ago by Russell Taylor
Re #6490. Put back SNS <archive> tag.
This is required for the archive search within the FileFinder. Although it uses ICAT underneath, it is unrelated to the Catalog menu.
Changeset: 9d37d0a719bb7dc759f50cf879261e9db7d21f70
comment:17 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 7336
handle a lack of catalog at a facility better