포럼
처음으로 Pull Request를 해봤습니다... (SRL 문제 해결법 제공)
2014.03.30 21:14
처음으로 github에 문제를 수정하여 Pull Request를 해봤네요.
어딘가 잘못된 건 아닐까 싶습니다만.. ;ㅁ;
주소는 https://github.com/xpressengine/xe-core/pull/614 입니다.
--
srl을 빠르게 패치하고싶으신 분들이 계실겁니다.
document.controller.php 236번 줄에서
이거를 if(!$obj->document_srl) $obj->document_srl = getNextSequence();
이걸로 $obj->document_srl = getNextSequence();
바꿔줍니다.
그리고 comment.controller.php 295번 줄 주변에서
이거를 if(!$obj->comment_srl)
{
$obj->comment_srl = getNextSequence();
}
이걸로 $obj->comment_srl = getNextSequence();
바꿔줍니다.
이러면 임의의 srl 값을 넣어도 그 값으 사용하지 않습니다.
버그 발견. 수정하지 마십시요.
XE에다 공개한 자료만 30건, 누적 다운로드 13만회를 넘긴 잉여 개발자입니다.
atde.kr img.atde.kr file.atde.kr svet.kr
댓글 11
-
테스트해보겠습니다.
-
W.O
2014.03.30 21:18
글 수정시나 파일 첨부시에 문제가 발생하지 않을까요?
-
파일 첨부 문제는 file 모듈을 수정해야하는것으로 보고 지금 테스트를 거치며 수정중에 있습니다. :)
-
W.O
2014.03.30 21:27
으... 제가 원래 php쪽은 진짜 모르는데 이렇게 일요일 밤을 불태우네요.. 레이아웃 제작의뢰 하나를 주말에 마무리지어야하는데 말입니다..
제 경우에는 에디터모듈의 자동저장 문제를 들여다보고 있습니다. 첨부파일 못지 않은 문제네요.
간바레!
-
자동저장...문제도 있군요. ㅠ
-
KrteamENT
2014.03.31 00:32
자동저장 저거 저장만 하고 나중에 취소 잘못누르면 아주그냥 황천길 ㅇㅅㅇ;
-
KrteamENT
2014.03.30 21:27
file모듈에서 sequnce를 변경해야겠네요.
별도로 분리해서 그걸 가져오게끔 해야겠네요
-
지금 제 생각은 file 모듈에서 srl을 강제로 생성하게 한 후, 세션에 넣고 그 세션을 이용해 글쓰기 및 커멘트를 처리하는 방식입니다.
저 수정사항은 다시 수정해야겠네요.
-
Pull Request입니다~
-
엌 그렇군영...
https://github.com/xpressengine/xe-core/issues/274 이것도 한번 테스트 해보시겠어요?