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

두 가지 문제점 #1726

Closed
pat-al opened this issue Dec 17, 2014 · 13 comments
Closed

두 가지 문제점 #1726

pat-al opened this issue Dec 17, 2014 · 13 comments
Assignees
Milestone

Comments

@pat-al
Copy link

pat-al commented Dec 17, 2014

오랜 만에 새 판으로 판올림했는데, 여러 모로 아주 만족스럽고 모바일 스킨이 매우 마음에 듭니다.
문제점이 좀 있는데 고쳐 주셨으면 합니다.

1.10.1 베타2판을 써 보았는데, 덧글을 넣으려고 하면 이름 또는 본문을 넣으라는 말이 나와서 덧글을 넣을 수 없습니다. 그래서 1.10판으로 다시 돌아왔습니다. 지금 텍스트큐브의 공지 블로그도 같은 문제가 생기고 있어서 그 곳에 덧글을 남기지 못하고 이 곳에 글을 쓰고 있습니다.

펼기기/접기(more/less) 객체가 들어간 글은 웹 화면에 아무것도 전혀 뜨지 않습니다.
기본 스킨에 플러그인을 거의 쓰지 않는 상태에서 해 보아도 같습니다.

@achimnol
Copy link
Member

으...으아니;;
공지블로그에는 새 Google reCAPTCHA 플러그인이 적용되어 있고 저희 내부적으로 (당연히) 작동을 확인했습니다.
혹시 중국에서 접속 중이시라면 Google 쪽 서비스가 대부분 막혀있기 때문에 reCAPTCHA가 뜨지 않고 텍스트큐브 쪽에서는 reCAPTCHA 인증 정보를 요구해서 비슷한 상황이 발생할 수 있습니다. 댓글 달 때 오류메시지가 정확히 무엇인지 copy & paste 부탁드립니다.

현재 예상 원인은:

  • Illegal parameters => reCAPTCHA 플러그인이 필요로 하는 form field가 아예 없는 경우
  • Missnig reCAPTCHA response => reCAPTCHA 플러그인이 요구하는 form field가 존재하나 "로봇이 아닙니다" 체크를 하지 않은 경우 혹은 체크를 할 수 없었던 경우
  • 그 외의 경우 => 추가 디버깅 필요

펼치기/접기 기능은 jQuery 업데이트 또는 에디터 변경 사항과 관련있을 것으로 보입니다. 이건 따로 살펴보겠습니다.

@achimnol
Copy link
Member

또 한 가지, reCAPTCA 플러그인이 정상 작동하려면 스킨에서 각 글마다 달리는 코멘트 보여주기 링크(출력 시 onclick="loadComment(...); return false" 속성을 가진 a 태그)가 특정한 형식의 id 속성(id="[##_article_rep_rp_cnt_id_##]")을 가지고 있어야 정상적으로 인식·작동합니다. 현재 팥알님 블로그 스킨에는 이 부분이 적용되어 있지 않은데, 아래와 같이 수정한 후 다시 테스트 부탁드립니다.

<a id="[##_article_rep_rp_cnt_id_##]" href="#rp" onclick="[##_article_rep_rp_link_##]">
<s_rp_count> 덧글 <span class="cnt">[##_article_rep_rp_cnt_##]</span></s_rp_count>
</a>

이게 id 속성이 없어도 과거에는 이에 직접 의존하는 기능이 없어서 잘 동작했으나, 댓글 영역을 AJAX로 불러오는 순간 reCAPTCHA 플러그인이 동작하기 때문에 자바스크립트 후킹 지점을 알려주는 필수 요소가 되었습니다.
또한 장기적으로는 HTML5 지원 확대 과정에서 onclick, onsubmit처럼 HTML에 직접 들어가있는 자바스크립트들을 가능한 모두 빼낼 것이기 때문에 더욱 스킨이 정확하게 작성되어 있어야 migration 비용을 줄일 수 있습니다.

@pat-al
Copy link
Author

pat-al commented Dec 17, 2014

펼치기/접기 객체가 들어간 글 가운데 며칠 사이에 다른 분과 덧글을 주고 받은 적이 있는 텍스트큐브 플러그인 배포 글도 있어서 1.9.3으로 잠시 되돌렸습니다.
좀 전에 새로 갱신하신 듯한데, 말씀해 주신 스킨 부분도 수정해서 확인해 보겠습니다.
(텍스트큐브에서 고쳐 쓰는 부분이 있어서 판올림해 쓰는 작업이 좀 늦습니다.)

reCAPTCHA 플러그인은 일부러 켜지 않았고 국내에서 접속하고 있으므로, reCAPTCHA 플러그인 때문에 생기는 문제는 아닐 것 같습니다.

@pat-al
Copy link
Author

pat-al commented Dec 17, 2014

reCAPTCHA 플러그인을 쓰지 않는 상태에서 현재 github에 올라온 최신판으로 제 블로그 안에서 덧글을 넣으면,
손님 상태에서는 "이름을 입력해 주십시오."라고 메시지창이 뜹니다.
관리자 상태에서는 "본문을 입력해 주십시오"라고 뜹니다.
텍스트큐브 공지 블로그는 reCAPTCHA 그림이 보이지 않아서(엑박?) 덧글을 넣어 볼 수 없습니다.

펼치기/접기 객체가 들어간 글은 여전히 나오지 않습니다.
IE에서 "웹 사이트에서 페이지를 표시할 수 없습니다."라고 시작하는 하얀 화면만 나옵니다.

@pat-al
Copy link
Author

pat-al commented Dec 17, 2014

1.10 이상에서 펼치기/접기 객체는
[#M_more|less| _M#]
달랑 이것만 들어가도 하얀 오류 화면이 나옵니다.
플러그인이 거의 돌지 않는 초기 상태 블로그여도 그렇습니다.
플러그인에서 쓰는 치환자를 글에 집어 넣었을 때 비슷한 일을 겪고 있었습니다.
([##_SNS_icons_##]를 치환자로 쓰는 플러그인을 켜 놓고 본문에 [##_SNS_icons_##]를 넣으면 하얀 화면이 나와서 플러그인 설멍문을 만들 때 애를 먹음)

@inureyes
Copy link
Member

일단 댓글의 경우 플러그인을 끈 상태에서도 standard / periwinkle 스킨에서 정상적으로 동작하는 것을 확인했습니다.

@pat-al
Copy link
Author

pat-al commented Dec 17, 2014

1.10.1 베타2은 아직도 제가 설치한 블로그와 텍큐 공지 블로그 모두 덧글 내용을 다 채워도 '이름'을 요구하는데, 1.10 정식판은 이상이 없습니다.
1.10에 펼치기/접기가 고쳐진 부분만 덧씌우니 크게 불편한 점은 사라졌습니다.
확인해서 고쳐 주시고 좋은 정보도 가르쳐 주셔서 고맙습니다.

@pat-al
Copy link
Author

pat-al commented Dec 18, 2014

제가 다른 도구(phpbb)에서 도메인 주소 설정을 바꾸었다가 쿠키 때문에 이용자들이 글을 넣지 못하게 되는 현상을 일으켰는데, 쿠키 이름을 바꾸어서 해결한 적이 있습니다.
그걸 떠올리고 textcube.org 쿠키를 지웠더니 공지 블로그에 덧글이 잘 올라갑니다.

@inureyes
Copy link
Member

@pat-al 👍

@zvuc
Copy link
Contributor

zvuc commented Dec 26, 2014

more/less 글펼침이 들어간 페이지가 여전히 흰 페이지로 로딩이 안되는데, 이 문제 해결이 아직 안 된 것인가요?

@inureyes
Copy link
Member

@zvuc 1.10.1 RC1에서는 해결이 되어 있어야 합니다. 혹시 코드캐시를 사용중이시면 플러싱을 한 번 해주세요~

@zvuc
Copy link
Contributor

zvuc commented Dec 27, 2014

2014-12-27 기준 1.10 트리 최신소스 (RC2로 나오네요)로 덮어씌우니 more/less 기능은 문제는 해결이 되었네요. 감사합니다 :)

inureyes added a commit that referenced this issue Dec 31, 2014
 - prefix can be changed via config.php
 - default prefix is Textcube[VERSION_NUMBER_WITHOUT_PERIOD]
inureyes added a commit that referenced this issue Dec 31, 2014
 - prefix can be changed via config.php
 - default prefix is Textcube[VERSION_NUMBER_WITHOUT_PERIOD]
@achimnol
Copy link
Member

achimnol commented Jan 3, 2015

more/less 기능은 저도 같은 증상 있었는데 이제 잘 동작하는 것 확인했습니다.

@inureyes inureyes closed this as completed Jan 4, 2015
@inureyes inureyes added fixed and removed fixed labels Jan 4, 2015
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

4 participants