Ticket #4457 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Problems with updating with a case insensitive ADS

Reported by: Nick Draper Owned by: Martyn Gigg
Priority: major Milestone: Release 2.0
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Owen Arnold

Description

The ADS is currently case insensitive, but this is causing problems with graph updating and python interaction.

The ADS should be made case sensitive (like it used to be a while ago).

Test case 

a=Load("HET16030")
T=ExponentialCorrection(a,1,1.2)
t=SplineBackground(a)

Change History

comment:1 Changed 9 years ago by Martyn Gigg

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Martyn Gigg

Refs #4457. The DataService is now case insensitive again.

Retrieval will search through eact, UPPER, lower and Sentence case before saying it can't find anything.

Changeset: 8c12ea3112be87f6bed1b20f4dfe430d6b720eca

comment:3 Changed 9 years ago by Martyn Gigg

  • Status changed from accepted to verify
  • Resolution set to fixed

Commit message should say case sensitive!

comment:4 Changed 9 years ago by Owen Arnold

refs #4457. Fixes the issue for the 3D case.

Changeset: c9a8c4f16cf86dca4daf9ad42a3310fe18a0fe4f

comment:5 Changed 9 years ago by Owen Arnold

Last commit was meant for #4475. Sorry.

comment:6 Changed 9 years ago by Owen Arnold

  • Status changed from verify to verifying
  • Tester set to Owen Arnold

comment:7 Changed 9 years ago by Owen Arnold

  • Status changed from verifying to closed

I've done some ad-hoc testing of my own here and it is definitely the case via the c++ ADS API that case sensitivity is used.

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​5304

Note: See TracTickets for help on using tickets.