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

파비콘 등록 과정 수정 요망 #695

Closed
ForPeople opened this issue May 9, 2014 · 3 comments
Closed

파비콘 등록 과정 수정 요망 #695

ForPeople opened this issue May 9, 2014 · 3 comments

Comments

@ForPeople
Copy link
Contributor

파비콘을 등록할때 계속 icon 파일이 아니라고 떠서 install 모듈을 살펴보니
install.admin.controller.php 파일 339 라인에 잘못 설정되어 있어서 파비콘이 등록 안되었던 것 같습니다.

if(!preg_match('/^.(ico).$/',$type)) {
Context::set('msg', '*.ico '.Context::getLang('msg_possible_only_file'));
return;
}

이렇게 바꿔주면 좋을 것 같습니다.

ngleader added a commit that referenced this issue May 9, 2014
@ngleader ngleader added this to the 1.7.5 milestone May 9, 2014
@ngleader ngleader self-assigned this May 9, 2014
@ngleader ngleader closed this as completed May 9, 2014
@upgle
Copy link
Contributor

upgle commented May 16, 2014

해당 커밋을 롤백한다면 PULL-REQUEST한
#713 5906704 과 충돌할 것 같습니다.

98de6e9 커밋을 보니 확장자 필터로 정규식을 수정하신것 같은데 확장자로 검사하기때문에 정규식을 파일 타입이 아닌 파일 이름의 확장자를 검사하도록 커밋했습니다.

이름을 검사하는것과, 형식을 검사하는 것에 차이점이 있나요?

@ForPeople
Copy link
Contributor Author

뭘로 수정되던 코딩에는 정답이 정해져 있는게 아니므로 저는 오류가 안나면 만족합니다.

@anumana
Copy link

anumana commented May 25, 2014

또 다른 해결 방법은
upgle/xe-core@5906704
이곳에 나오네요.

@ghost ghost unassigned ngleader Feb 19, 2016
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

5 participants