테스트 사이트 - 개발 중인 베타 버전입니다

패치를 하는 중인데 Github내용이 좀 다른게 있어요. 채택완료

곰x3 4년 전 조회 1,466

5.4.2.3 버전을 깔아놨는데 패치를 이제사 인지하고 시작하는 중인데요.

많이는 아니지만 코어를 건들인것도 있고 테마는 아예 수정해서 덮어쓰지는 못하고 Github와 패치파일과 제 파일을 비교해서 보는 중이에요.

그런데 GitHub에는 A라고 되어있는데 패치파일에는 B라고 되어있어서 앞으로 해야할 패치파일을 다 봐도 B라고 되어있고 그누보드 5.4.15.1버전에도 B라고 되어있어요.

 

get_group 함수 실행시 string 인지 체크 이 부분을 패치하다가 발견했는데

https://github.com/gnuboard/gnuboard5/commit/1cc2b141f954fb3fa8ff966746f965caca99722b

 

</p>

<p>if ($gr_id) {

<strong>if ($gr_id && is_string($gr_id)) {</strong>

    $group = get_group($gr_id);

}</p>

<p>

 

고치라고 되어있는 아래의 코드가

if ($gr_id && is_string($gr_id)) {

모든 파일에서 아래처럼 되어있어요.

if ($gr_id && !is_array($gr_id)) {

 

이럴 경우엔 Github에 올려둔 내용이 잘못 된것인거죠?

댓글을 작성하려면 로그인이 필요합니다.

답변 2개

채택된 답변
+20 포인트
그누위즈

버그 신고에 문의해보시는게 나을듯합니다.

https://sir.kr/cm_bug?sca=%EA%B7%B8%EB%88%84%EB%B3%B4%EB%93%9C5

로그인 후 평가할 수 있습니다

답변에 대한 댓글 1개

곰x3
4년 전
아! 감사합니다. 올려볼게요.

댓글을 작성하려면 로그인이 필요합니다.

if 쪽과 else 쪽을 서로 바꾼 거 아닐까요?

로그인 후 평가할 수 있습니다

답변에 대한 댓글 1개

곰x3
4년 전
같은 버전 패치 아래쪽에 다시 수정이 되더라구요.
위에서부터 처리해가다가 막혀서 밑을 볼 생각을 못했어요.;;;

댓글을 작성하려면 로그인이 필요합니다.

답변을 작성하려면 로그인이 필요합니다.

로그인