[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2020-01-14 Thread gerritbot
gerritbot added a comment.


  Change 563958 abandoned by Addshore:
  wbterms: Update SingleEntitySourceServices PrefetchingTermLookup
  
  Reason:
  Abandoning as I figure we are not backporting this!
  
  https://gerrit.wikimedia.org/r/563958

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: Addshore, gerritbot
Cc: sarhan.alaa, Lydia_Pintscher, hoo, Addshore, Ladsgroup, 
Lucas_Werkmeister_WMDE, Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 
0010318400, E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2020-01-14 Thread gerritbot
gerritbot added a comment.


  Change 553524 **merged** by jenkins-bot:
  [mediawiki/extensions/Wikibase@master] Validate ChangeOpResult in ModifyEntity
  
  https://gerrit.wikimedia.org/r/553524

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: Addshore, gerritbot
Cc: sarhan.alaa, Lydia_Pintscher, hoo, Addshore, Ladsgroup, 
Lucas_Werkmeister_WMDE, Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 
0010318400, E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2020-01-13 Thread gerritbot
gerritbot added a comment.


  Change 563958 had a related patch set uploaded (by Ladsgroup; owner: 
Addshore):
  [mediawiki/extensions/Wikibase@wmf/1.35.0-wmf.14] wbterms: Update 
SingleEntitySourceServices PrefetchingTermLookup
  
  https://gerrit.wikimedia.org/r/563958

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: Addshore, gerritbot
Cc: sarhan.alaa, Lydia_Pintscher, hoo, Addshore, Ladsgroup, 
Lucas_Werkmeister_WMDE, Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 
0010318400, E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2020-01-09 Thread gerritbot
gerritbot added a comment.


  Change 562320 **merged** by jenkins-bot:
  [mediawiki/extensions/Wikibase@master] wbterms: Update 
SingleEntitySourceServices PrefetchingTermLookup
  
  https://gerrit.wikimedia.org/r/562320

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: Addshore, gerritbot
Cc: sarhan.alaa, Lydia_Pintscher, hoo, Addshore, Ladsgroup, 
Lucas_Werkmeister_WMDE, Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 
0010318400, E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2020-01-09 Thread gerritbot
gerritbot added a comment.


  Change 563204 abandoned by Addshore:
  Update SingleEntitySourceServices to respect item terms migration stages
  
  https://gerrit.wikimedia.org/r/563204

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: Addshore, gerritbot
Cc: sarhan.alaa, Lydia_Pintscher, hoo, Addshore, Ladsgroup, 
Lucas_Werkmeister_WMDE, Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 
0010318400, E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2020-01-09 Thread gerritbot
gerritbot added a comment.


  Change 563204 had a related patch set uploaded (by Addshore; owner: Addshore):
  [mediawiki/extensions/Wikibase@master] Update SingleEntitySourceServices to 
respect item terms migration stages
  
  https://gerrit.wikimedia.org/r/563204

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: Addshore, gerritbot
Cc: sarhan.alaa, Lydia_Pintscher, hoo, Addshore, Ladsgroup, 
Lucas_Werkmeister_WMDE, Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 
0010318400, E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2020-01-07 Thread Addshore
Addshore added a comment.


  Added the 2 things we found as sub tasks as they need to be done to be able 
to test this code in WRITE_NEW only

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: Addshore
Cc: sarhan.alaa, Lydia_Pintscher, hoo, Addshore, Ladsgroup, 
Lucas_Werkmeister_WMDE, Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 
0010318400, E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2020-01-06 Thread sarhan.alaa
sarhan.alaa added a comment.


  Oh, apparently `SpecialNewEntity` creates entities without even creating 
change ops. That would explain it, as then it will by-pass the whole validation 
of fignreprint uniqueness that is only executed inside 
`ChangeOpFingerprintResult::validate()`. Guess that needs to change too.

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: sarhan.alaa
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, CucyNoiD, 
Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, Giuliamocci, 
Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, Lordiis, 
GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2020-01-06 Thread gerritbot
gerritbot added a comment.


  Change 562320 had a related patch set uploaded (by Sarhan; owner: Sarhan):
  [mediawiki/extensions/Wikibase@master] Update SingleEntitySourceServices to 
respect item terms migration stages
  
  https://gerrit.wikimedia.org/r/562320

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: sarhan.alaa, gerritbot
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, CucyNoiD, 
Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, Giuliamocci, 
Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, Lordiis, 
GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2020-01-06 Thread sarhan.alaa
sarhan.alaa added a comment.


  Same goes for items. API calls fail manually as expected, but I can create 
conflicting items through special pages. I also confirm that 
`SingleEntitySourceServices::getTermSearchInteractorFactory` and 
`SingleEntitySourceServices::getPrefetchingTermLookup` need fixing. Will be 
working on that this evening.
  
  note: Seems like that API calls somehow are configure to respect federation, 
using `PerReopsitoryServiceWiring` that was updated to respect item migration 
phase too in 
https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Wikibase/+/554562, but 
Special Page would use `SingleEntitySourceServices`. Though conflicting 
property creation should have failed as `SingleEntitySourceServices` was 
updated to respect property migration stage. So there could be more buggy code 
down the call stack.

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: sarhan.alaa
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, CucyNoiD, 
Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, Giuliamocci, 
Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, Lordiis, 
GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2020-01-06 Thread sarhan.alaa
sarhan.alaa added a comment.


  I added two test cases to the integration test editentity fingerprint 
uniqueness. Those pass in for API calls, which I could also confirm sending API 
requests manually.
  
  But, I can also reproduce it through special pages: I can create conflicting 
properties (didn't test items). I will next investigate the mentioned 
`SingleEntitySourceServices::getTermSearchInteractorFactory` and 
`SingleEntitySourceServices::getPrefetchingTermLookup`.

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: sarhan.alaa
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, CucyNoiD, 
Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, Giuliamocci, 
Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, Lordiis, 
GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2020-01-03 Thread Addshore
Addshore added a comment.


  While reviewing 
https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Wikibase/+/553524/ I 
noticed that both at least 2 more things need fixing:
  
  - SingleEntitySourceServices::getTermSearchInteractorFactory
  - SingleEntitySourceServices::getPrefetchingTermLookup

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: sarhan.alaa, Addshore
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, CucyNoiD, 
Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, Giuliamocci, 
Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, Lordiis, 
GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2019-12-30 Thread Ladsgroup
Ladsgroup added a comment.


  I'm on vacation until next Monday.

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: sarhan.alaa, Ladsgroup
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, CucyNoiD, 
Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, Giuliamocci, 
Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, Lordiis, 
GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2019-12-30 Thread sarhan.alaa
sarhan.alaa added a comment.


  If reviewing the patch in the current state is too much effort or difficult 
(it can be very much the case), also let me know I can try to split it up 
further into couple of smaller patches

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: sarhan.alaa
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, CucyNoiD, 
Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, Giuliamocci, 
Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, Lordiis, 
GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2019-12-30 Thread sarhan.alaa
sarhan.alaa added a comment.


  @addshore @ladsgroup
  We are still on the plan to swit

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: sarhan.alaa
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, CucyNoiD, 
Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, Giuliamocci, 
Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, Lordiis, 
GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2019-12-19 Thread sarhan.alaa
sarhan.alaa added a comment.


  Not sure @Ladsgroup your comments in the review do make sense.. can you 
please double check them and read through the code again? I may of course have 
misunderstood them, maybe little more elaboration will help

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: sarhan.alaa
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, CucyNoiD, 
Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, Giuliamocci, 
Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, Lordiis, 
GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2019-12-12 Thread sarhan.alaa
sarhan.alaa added a comment.


  Last patch turning on validation of uniqueness in new store is ready for 
review now
  https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Wikibase/+/553524
  It has 3 little fixes below it too

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: sarhan.alaa
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, DannyS712, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2019-12-10 Thread gerritbot
gerritbot added a comment.


  Change 553213 **merged** by jenkins-bot:
  [mediawiki/extensions/Wikibase@master] Add Fingerprint uniqueness validation 
based on new term store
  
  https://gerrit.wikimedia.org/r/553213

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: sarhan.alaa, gerritbot
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, DannyS712, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2019-12-09 Thread gerritbot
gerritbot added a comment.


  Change 556063 abandoned by Sarhan:
  Add Fingerprint uniqueness validation based on new term store
  
  Reason:
  duplicate
  
  https://gerrit.wikimedia.org/r/556063

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: alaa_wmde, gerritbot
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, DannyS712, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2019-12-09 Thread gerritbot
gerritbot added a comment.


  Change 556063 had a related patch set uploaded (by Sarhan; owner: Alaa 
Sarhan):
  [mediawiki/extensions/Wikibase@master] Add Fingerprint uniqueness validation 
based on new term store
  
  https://gerrit.wikimedia.org/r/556063

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: alaa_wmde, gerritbot
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, DannyS712, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2019-12-09 Thread gerritbot
gerritbot added a comment.


  Change 546898 **merged** by jenkins-bot:
  [mediawiki/extensions/Wikibase@master] Add TermsCollisionDetector and db 
implementations for properties and items
  
  https://gerrit.wikimedia.org/r/546898

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: alaa_wmde, gerritbot
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, DannyS712, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2019-12-03 Thread gerritbot
gerritbot added a comment.


  Change 552905 **merged** by jenkins-bot:
  [mediawiki/extensions/Wikibase@master] Use ChangeOpFingerprint to wrap 
fingerprint (terms) change operations
  
  https://gerrit.wikimedia.org/r/552905

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: alaa_wmde, gerritbot
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, DannyS712, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2019-12-03 Thread gerritbot
gerritbot added a comment.


  Change 550730 **merged** by jenkins-bot:
  [mediawiki/extensions/Wikibase@master] Add ChangeOpFingerprint and 
ChangeOpFingerprintResult classes
  
  https://gerrit.wikimedia.org/r/550730

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: alaa_wmde, gerritbot
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, DannyS712, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2019-11-28 Thread gerritbot
gerritbot added a comment.


  Change 553524 had a related patch set uploaded (by Alaa Sarhan; owner: Alaa 
Sarhan):
  [mediawiki/extensions/Wikibase@master] Validate ChangeOpResult in EditEntity
  
  https://gerrit.wikimedia.org/r/553524

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: alaa_wmde, gerritbot
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, DannyS712, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2019-11-27 Thread gerritbot
gerritbot added a comment.


  Change 550703 **merged** by jenkins-bot:
  [mediawiki/extensions/Wikibase@master] Add ChangeOpResult::validate
  
  https://gerrit.wikimedia.org/r/550703

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: alaa_wmde, gerritbot
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, DannyS712, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2019-11-26 Thread gerritbot
gerritbot added a comment.


  Change 553213 had a related patch set uploaded (by Alaa Sarhan; owner: Alaa 
Sarhan):
  [mediawiki/extensions/Wikibase@master] Add Fingerprint uniqueness validation 
based on new term store
  
  https://gerrit.wikimedia.org/r/553213

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: alaa_wmde, gerritbot
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, DannyS712, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2019-11-25 Thread gerritbot
gerritbot added a comment.


  Change 552905 had a related patch set uploaded (by Alaa Sarhan; owner: Alaa 
Sarhan):
  [mediawiki/extensions/Wikibase@master] [WIP] Use ChangeOpFingerprint to wrap 
fingerprint change operations wherever created.
  
  https://gerrit.wikimedia.org/r/552905

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: alaa_wmde, gerritbot
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, DannyS712, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2019-11-13 Thread gerritbot
gerritbot added a comment.


  Change 550730 had a related patch set uploaded (by Alaa Sarhan; owner: Alaa 
Sarhan):
  [mediawiki/extensions/Wikibase@master] Add ChangeOpFingerprint and 
ChangeOpFingerprintResult classes
  
  https://gerrit.wikimedia.org/r/550730

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: alaa_wmde, gerritbot
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, DannyS712, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2019-11-13 Thread gerritbot
gerritbot added a comment.


  Change 550703 had a related patch set uploaded (by Alaa Sarhan; owner: Alaa 
Sarhan):
  [mediawiki/extensions/Wikibase@master] Add ChangeOpResult::validate
  
  https://gerrit.wikimedia.org/r/550703

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: alaa_wmde, gerritbot
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, DannyS712, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2019-11-12 Thread Lydia_Pintscher
Lydia_Pintscher added a comment.


  Yeah it's not a super strict integrity check but if we can avoid Items and 
Properties with the same label/description combination we really should try.

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: alaa_wmde, Lydia_Pintscher
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, DannyS712, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2019-11-12 Thread alaa_wmde
alaa_wmde added a comment.


  True. Option 1 is the status quo, and since those race-conditions are quite 
rare to begin with, and the point made by @Lucas_Werkmeister_WMDE about this 
validation not being a strict integrity check (@Lydia_Pintscher please correct 
us if we are making wrong assumptions here) sounds reasonable, I will continue 
with option 1 for now.
  
  Thanks for the input!

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: alaa_wmde
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, DannyS712, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T232040: Add label and description collision detectors for new terms store

2019-11-12 Thread Ladsgroup
Ladsgroup added a comment.


  As I mentioned in person, given that DefferedUpdates happen practically at 
the same time of the edit, I'm fine with 1. If it was done in a job, I would be 
against it.

TASK DETAIL
  https://phabricator.wikimedia.org/T232040

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: alaa_wmde, Ladsgroup
Cc: Lydia_Pintscher, hoo, Addshore, Ladsgroup, Lucas_Werkmeister_WMDE, 
Aklapper, alaa_wmde, Hook696, Daryl-TTMG, RomaAmorRoma, 0010318400, 
E.S.A-Sheild, Iflorez, darthmon_wmde, Meekrab2012, joker88john, DannyS712, 
CucyNoiD, Nandana, NebulousIris, Gaboe420, Versusxo, Majesticalreaper22, 
Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Af420, Darkminds3113, Bsandipan, 
Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, 
LawExplorer, WSH1906, Lewizho99, Maathavan, _jensen, rosalieper, Scott_WUaS, 
Jonas, Wikidata-bugs, aude, Mbch331
___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs