파비콘 이슈 해결 및 가상사이트 대응 #713
Merged
파비콘 이슈 해결 및 가상사이트 대응 #713
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Develop merge from xpressengine:develop
등록 버튼 클릭 시 실제 등록되지 않고 install 모듈의 saveIconTmp 함수를 통해 임시 저장되어 미리보여주는 기능을 수행함. 따라서 오해의 소지가 있어 미리보기 버튼으로 수정.
정규식 검사에서 업로드된 file의 타입을 검사함으로서 파일이 등록되지 않는 이슈. 정상적인 ico 파일 업로드시 '*.ico 파일만 업로드 가능합니다.' 라는 오류 출력 tmp file의 'name'을 검사하도록 변경하여 이슈 해결
파비콘 저장시
files/attach/xeicon/{가상사이트srl}/favicon.ico
와 같이 저장하도록 개선
이전 버전과 호환을 위해 기본 사이트의 경우 files/attach/xeicon/favicon.ico 와 같이 기존 주소를 그대로
이용하도록 처리.
|
👍 |
Closed
akasima
added a commit
that referenced
this pull request
Jul 3, 2014
* upgle-develop-favicon: 파비콘이 가상 사이트에 대응하도록 개선 파비콘이 업로드 되지 않는 이슈 해결 파비콘 등록 버튼 이름 변경 파비콘 파일 등록 INPUT 타이틀 수정
ghost
modified the milestones:
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Xpressengine 파비콘 가상사이트 대응 및 기타 이슈 해결