ax5ui / ax5ui-grid

Javascript UI Component - GRID ( Excel Grid, jqGrid, angularjs grid, jquery grid, SlickGrid, ag-grid gridify)
http://ax5.io/ax5ui-grid/demo/index.html
MIT License
99 stars 36 forks source link

grid formatter money 버그... #76

Open ohHyunJae opened 7 years ago

ohHyunJae commented 7 years ago

안녕하세요

아래 코드와 같이

{ key: "price", label: "가격 (입력란)", formatter : "money", editor : { type : "money" }, align : "right" }

주고 해당 곳에 입력을 하면 음수일 경우

,가 붙는 순간 +로 변해버리는 버그가 있습니다. ( ,가 붙지 않을때는 - 잘 표기가 됩니다 )

확인부탁드리겠습니다. ( 최근 걸로 git에서 다운받은 소스에서 js는. . )

image

위 화면에 대한 아래..

image

thomasJang commented 7 years ago

패치된 버전으로 테스트 해주세요

ohHyunJae commented 7 years ago

안녕하세요

일이 있어서 확인해보지 못하다가 이제야 보내요 먼저 답변달아주신것에 감사드리며

해당 내용 다운받아서 해봐도 그대로더 군요

그래서 디버깅을 해서 확인 결과 ax5core.js 내 number 함수를 호출을 하는데

그 함수 내에서 아래 내용 때문에 안되었던거 같네요.

isMinus = Number(pair[0]) < 0 || pair[0] == "-0", 를 체크하는 곳 중 오류

isMinus가 true면 -붙이고 아니면 -를 안 붙이는데

위 처럼 가정하였을 때 숫자가 4자리 이상이 되면 str 매개변수에 ,가 붙어 있는 상태로 오는데

이때 잘 아시겠지만.. Number(pair[0]) 함수의 결과가 Nan이 됩니다.

core쪽에서 number 함수를 다른 util에서 어떻게 사용하는지 몰라 일단 저는 임시방편으로 core쪽 소스 isMinus = Number(pair[0].replace(/,/g, "")) < 0 || pair[0] == "-0",로 수정을 하였는데 ( 콤마만 replace를 해주었죠.. )

확인해주시고 수정한 버전 올려주시면 감사하겠습니다.