Addshore added a comment.

  When trying to remove things I ran into an issue with 
  It seems some of the stuff in T96553 
<> is related.
  It seems the old EntityInfo based lookup stuff is used in this integration 
test, (even if it is not used in production).
  The situation being tested is...
  - Q1 redirects to Q2.
  - Q2 has a label of "foo"
  - Q3 has a statement using Q1 as a value
  - The expected behaviour is that the value uses the label from Q2 (redirect 
target) even though the value on the entity is still Q1
  Checking on this is currently the case
  Checking locally (with the patch removing entityinfo from view classes) this 
is also the case... just the test is evil?



To: Addshore
Cc: Jakob_WMDE, Addshore, Aklapper, Blissjay007, Oblanco79, Alter-paule, 
Beast1978, Un1tY, Hook696, Daryl-TTMG, RomaAmorRoma, E.S.A-Sheild, 
darthmon_wmde, Kent7301, Meekrab2012, joker88john, DannyS712, CucyNoiD, 
Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, Giuliamocci, 
Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, Lordiis, 
Pablo-WMDE, GoranSMilovanovic, Adik2382, Jayprakash12345, Th3d3v1ls, Ramalepe, 
Liugev6, QZanden, LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, 
rosalieper, Scott_WUaS, Izno, Wikidata-bugs, aude, Dinoguy1000, Mbch331, Jay8g
Wikidata-bugs mailing list

Reply via email to