search

Needlworks / Textcube

Textcube : Brand yourself! / Personalized web publishing platform with multi-user support
http://www.textcube.org
Other
207 stars 55 forks source link

옵션에서 스마트폰용 인터페이스 옵션을 체크 해제해도 완전히 해당 기능이 비활성화되지 않음 #1856

Closed zvuc closed 7 years ago

zvuc commented 7 years ago
image image

옵션에서 해당 항목을 체크 해제하면, 자동으로 모바일 전용 페이지 (/i)로 이동되지는 않지만 여전히 주소로 입력해 들어가면 접속이 가능합니다. 스타일시트가 깨진 채로 페이지가 표시되어 구글 등에서 크롤링할때 이 주소로 된 페이지까지 크롤링되어 본 사이트의 스킨이 모바일 최적화가 완벽히 된 경우에도 /i, /m URL의 페이지들이 에러로 표시되어 'Mobile-friendly' 마크를 못 받게되는 문제가 있습니다.

체크를 해제한 경우에는 주소로 접속해도 메인으로 리다이렉트 되도록 완전히 기능을 해제할 필요가 있을것 같습니다.

zvuc commented 7 years ago

조금 더 디깅을 해보니, 정확한 시점은 잘 모르겠지만 1.10.9나 1.10.10로 올라가는 도중에 이전 버전에는 있던 library/view/iPhoneView.php가 사라졌는데 업그레이드를 하면서 기존 파일이 삭제되지 않아서 그대로 html출력이 되었던것 같습니다. 다만 그래도 여전히 /i주소를 직접 치고 들어가면 백지인 페이지가 뜬다는 문제가 있고, /m의 경우에는 정말 이상하게도 /m 페이지는 무효화할수 있지만 /m/comment/1이런식으로 코멘트를 출력하던 주소를 들어가면 여전히 페이지가 출력이 됩니다. 로컬 환경에서 동일한 1.10 브랜치 최신 코드로 구동을 해보았는데 여기서는 해당 주소로 들어가면 아무것도 안 나오네요.

정확히 어느 부분이 문제가 되는걸까요?

zvuc commented 7 years ago

버전업이 되면서 삭제된 interface/i/, interface/m/ 폴더가 그대로 남아있어서 페이지 출력이 되었던것같네요. 폴더를 삭제하니 제대로 그냥 404로 뜹니다.

이 이슈는 닫겠습니다만, /i 나 /m 등 실재하지 않는 URL로 접속시 텍큐 내장의 404 페이지를 보여주는 식으로 처리를 하는 것은 필요할것 같습니다.