search

serkandurusoy / mt.issues

Mitolojix issue tracker
0 stars 0 forks source link

Isme gore siralamalarda sorun var #541

Open serkandurusoy opened 7 years ago

serkandurusoy commented 7 years ago

@cyardimci

Kurum ismi, kullanici ismi gibi siralamalara bakar misin (yeni kayit ekleyerek) turkce karakterle baslayanlar en sona dusuyor sanki. Birden dikkatimi cekti ayrintili bir test yapmadim.

Ama eger oyleyse detayina bakmak gerekecek.

Lokalinde de test et lutfen. Veritabanindan da ayrica bak. Misal kurum icin isim sahasinin bir de isimCollate karsiligi olmasi lazim, isim'den turetilen bir hash olacak o sahanin icinde. isme gore dizdiginde turkce karakter sonda gelirken,o hash'e gore dizdiginde turkce yazim kurallarina gore siralanmasi gerekiyor. Eger degilse, o hash'leri olusturan kutuphanede sorun var demektir. Tabi eger hash hic yoksa baska, o zaman bizim kodda sorun var demektir.

cc @BirolU @muratulman bu vesileyle, testleri aman lutfen yogun ve ayrintili yapalim, gorunurde calisiyor gibi olabilir ama seytan ayrintida gizli.

muratulman commented 7 years ago

Son eklenen hep en sona düşüyor ki zaten...

muratulman commented 7 years ago

Üzerinde değişiklik yapılan kayıtlar da en sona düşüyor. Hiçbir zaman alfabetik bir sıralama olmadı ki.

serkandurusoy commented 7 years ago

Vardi kullanicilara vb bir bak istersen, alfabetik siralama vardi hep.

sorular ve sinavlara bakma, onlarda yoktu cunku onlarin farkli siralama oncelikleri var.

On Wed, Jan 11, 2017 at 1:58 PM, muratulman notifications@github.com wrote:

Üzerinde değişiklik yapılan kayıtlar da en sona düşüyor. Hiçbir zaman alfabetik bir sıralama olmadı ki.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/mitolojix/issues/issues/541#issuecomment-271839690, or mute the thread https://github.com/notifications/unsubscribe-auth/AEbz3MUfEKv4mazpoLDDkrvb-0LwOFrdks5rRLXLgaJpZM4Lgbwe .

cyardimci commented 7 years ago

@serkandurusoy master ve developta karsilastirmalı test yaptım. Sanirim hash yaratma algoritmalarında dedigin gibi degisiklige gitmisler. Master'da uretilen hashler (turkce karakterden bagımsız) full numeric bir pattern ile giderken developta uretilen hashlerde arada char da var. Ornek vermek gerekirse developta Çağrı isminin name collatei "nameCollate" : "c76111f72131" iken, masterda Çağrı isminin name collate'i "nameCollate" : "303630303032313032326132313432".

Master branchında alfabetik olarak Çağrı'dan önce gelen 'Cagr' kelimesinin name collatei: "nameCollate" : "303430303032306532326132", Çağrı'dan sonra gelen 'Deniz' isminin name collatei "nameCollate" : "303830306132323032313632336532". Yani siralama su sekilde: Cagr: "303430303032306532326132" Çağrı: "303630303032313032326132313432" Deniz: "303830306132323032313632336532"

Develop brach hash algoritmasında ise 'C' harfi sanırım 43 'D' harfi de 44 ile sifrelenmis. 'Ç' icin arada bir yer yok, ona direk 'c7' atamıslar.

Yani sıralama olarak: C... : "43...." D... : "44...." Ç...: "c7...."

serkandurusoy commented 7 years ago

ok bu veriler CLDR data denen global bir kutuphaneden aliniyor, kullandigimiz library'de onu kullaniyor, ve bu bahsettigin vurgular, biz locale olarak ne verirsek verelim (tr veriyoruz) en_US kabul edildigini gosteriyor kanimca.

Tabi master ve develop arasinda bu kodda bir degisiklik sozkonusu olmadi.

Bu da demektir ki, module system ile ilgili bir problem var (node versiyonu da degisti dogal olarak).

Kutuphaneyi disarida node'de standalone deneyip sonuca bakmak (ki kutuphane neredeyse 2 yildir guncelleme almiyor gorunuyor) eger node'da da calismazsa, kutuphanenin kendi module bundler'inda sorun var diyerek ya fork edip hatayi duzeltecegiz ya da idareten sadece TR destekli (CLDR datasini kopyalayip) kendimiz bir sey yapacagiz.

bu durumda bu issue'yu bana assign edip birakabilirsin.

tesekkurler

2017-01-11 18:59 GMT+03:00 cyardimci notifications@github.com:

@serkandurusoy https://github.com/serkandurusoy master ve developta karsilastirmalı test yaptım. Sanirim hash yaratma algoritmalarında dedigin gibi degisiklige gitmisler. Master'da uretilen hashler (turkce karakterden bagımsız) full numeric bir pattern ile giderken developta uretilen hashlerde arada char da var. Ornek vermek gerekirse developta Çağrı isminin name collatei "nameCollate" : "c76111f72131" iken, masterda Çağrı isminin name collate'i "nameCollate" : "303630303032313032326132313432".

Master branchında alfabetik olarak Çağrı'dan önce gelen 'Cagr' kelimesinin name collatei: "nameCollate" : "303430303032306532326132", Çağrı'dan sonra gelen 'Deniz' isminin name collatei "nameCollate" : " 303830306132323032313632336532". Yani siralama su sekilde: Cagr: "303430303032306532326132" Çağrı: "303630303032313032326132313432" Deniz: "303830306132323032313632336532"

Develop brach hash algoritmasında ise 'C' harfi sanırım 43 'D' harfi de 44 ile sifrelenmis. 'Ç' icin arada bir yer yok, ona direk 'c7' atamıslar.

Yani sıralama olarak: C... : "43...." D... : "44...." Ç...: "c7...."

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mitolojix/issues/issues/541#issuecomment-271907886, or mute the thread https://github.com/notifications/unsubscribe-auth/AEbz3DuCtgwMySUOiQd5j7qfD9EVSs1uks5rRPxtgaJpZM4Lgbwe .

serkandurusoy commented 7 years ago

@muratulman bunu ios issue'suna gecmeden once diger issular kapandiktan sonra yapacagiz zira tum veritabaninin uzerinden bir pas gecip eski collation verilerini yeni algoritmayla yeniden uretip guncellemek gerekecek. altyapisi benim tarafimda tamam, veritabani operasyonunu diger issuelar bitsin sunucu yuklemesi oncesi @cyardimci ile yapacagiz