Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1.7.4 - 쪽지함에서 전체 선택이 안 되는 버그 (패치제공) #471

Closed
sejin7940 opened this issue Feb 12, 2014 · 3 comments
Closed
Assignees
Milestone

Comments

@sejin7940
Copy link
Contributor

#468
의 이슈와 같은 건데..
쪽지목록 등 여러 상황에서 checkbox 한꺼번에 선택되는 현상 패치입니다.
XE.checkboxToggleAll 부분 script 오류입니다.

실제 예전 버전과 소스 자체는 달라진게 없는듯한데 XE 1.7.4 에서 안 되는걸보면
jquery 쪽 버전이 올라가면서 문제가 생긴듯한데....

common/js/xe.min.js 파일에서

find("input[name="+b+"]:checkbox"):a("input[name="+b+"]:checkbox")
를 아래걸로 교체

find("input[name='"+b+"']:checkbox"):a("input[name='"+b+"']:checkbox")

common/js/xe.js 파일기준으로는

58번째줄에

if(options.wrap) {
obj = $(options.wrap).find('input[name='+itemName+']:checkbox');
} else {
obj = $('input[name='+itemName+']:checkbox');
}

를 아래걸로 교체

if(options.wrap) {
obj = $(options.wrap).find('input[name="'+itemName+'"]:checkbox');
} else {
obj = $('input[name="'+itemName+'"]:checkbox');
}

@bjrambo
Copy link
Contributor

bjrambo commented Feb 12, 2014

!!! 👍 감사합니다

@akasima
Copy link
Member

akasima commented Feb 13, 2014

classes/display/HTMLDisplayHandler.php 에서 확인해 보니 디버그 모드에서 불러오는 파일에 xe.js 는 없네요.

common.js 62, 64 line 을 수정했습니다.

xe.min.js 파일은 따로 준비해서 커밋하겠습니다.

@bjrambo
Copy link
Contributor

bjrambo commented Feb 13, 2014

#468 이슈는 닫았습니다. 여기서 처리 하는게 낫겠네요 :)

@akasima akasima added this to the 1.7.5 milestone Feb 13, 2014
akasima added a commit that referenced this issue Feb 13, 2014
@akasima akasima closed this as completed Feb 13, 2014
@ghost ghost added the type/bug label Feb 26, 2014
@ghost ghost assigned akasima Feb 26, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants