Commit graph

16236 commits

Author SHA1 Message Date
Kijin Sung
2cca84bd3f Follow XE version 1.9.0 2017-12-01 02:09:15 +09:00
Kijin Sung
5d29d66041 Allow PHP 7.2 test failures for now 2017-12-01 01:57:11 +09:00
Kijin Sung
3992b4b5bc Fix strict count() behavior in PHP 7.2 2017-12-01 01:49:20 +09:00
Kijin Sung
6f35f5bafc Fix strict count() behavior in PHP 7.2 2017-12-01 01:42:03 +09:00
Kijin Sung
6f7f92f3e2 Add PHP 7.2 to test suite 2017-12-01 01:19:53 +09:00
Kijin Sung
7cfc23eedb
Merge pull request #947 from bjrambo/pr/changeObject
php7.2호환을 위해 변경한 클래스 Object 를 BaseObject으로 변경
2017-12-01 01:18:22 +09:00
Kijin Sung
9c540ea414 Fix classes inheriting Object 2017-12-01 01:14:23 +09:00
Kijin Sung
ae971306b7 Change instanceof Object to instanceof BaseObject 2017-12-01 01:10:48 +09:00
Kijin Sung
84e5542d77 Remove unnecessary use of BaseObject
- 트리거 등 반환값이 필요하지 않은 곳에서 new BaseObject()를 반환하는 것 삭제
- 모듈 설치, 업데이트 후 무의미한 new BaseObject()를 반환하는 것 삭제
- 사용자에게 에러 메시지를 돌려주는 용도로 new BaseObject(-1, '에러메시지')를
  사용하는 경우는 대부분 $this->setError()로 변경함. 언어 변환과 sprintf()
  처리까지 한 번에 이루어지므로 이쪽이 더 편리함.
2017-12-01 00:54:51 +09:00
Kijin Sung
03d74a984f Merge branch 'develop' into pr/change-object 2017-11-30 23:30:21 +09:00
Kijin Sung
e19c196a67 Fix too many simultaneous requests on agreement edit screen
약관 에디터 iframe을 동시에 5개 띄우면서 과부하가 발생하는 문제 수정
2017-11-30 23:23:07 +09:00
Kijin Sung
04fbcdd514 Fixed single quotes in category XML output 2017-11-30 23:03:45 +09:00
Kijin Sung
2dbc77ff68 Eliminate null returns in DB class 2017-11-30 23:03:18 +09:00
Kijin Sung
e48179aa05 다국어 등 코드를 생성하여 만들어지는 cache 파일의 취약점 수정
- 관리자 메뉴, 사용자 지정 다국어 cache를 object cache로 변경
- xpressengine/xe-core#2182
2017-11-30 22:50:36 +09:00
Kijin Sung
999bc92753 mailto: 등 URN 링크 문제 고침 by @bnu
xpressengine/xe-core#2174
2017-11-30 22:11:55 +09:00
bnu
a199333a2e fix #2177 XEVE-17-023 회원의 이메일을 회원 정보에서 노출하지 않도록 변경
- 모듈 업데이트를 통해 기존 설정이 변경됩니다
- 회원의 이메일 주소를 공개하도록 설정할 수 없도록 합니다
- 최고 관리자에게는 회원 정보보기에서 이메일 주소를 확인할 수 있습니다
2017-11-30 22:03:10 +09:00
Kijin Sung
d7beeb783c config파일 읽을 때 BOM문자 제거
xpressengine/xe-core#1604
2017-11-30 21:53:23 +09:00
BJRambo
1803883e1d 코드가 new Object으로만 작성된 코드에 대해서도 동일하게 BaseObject 호출하도록 추가. 2017-11-28 20:07:08 +09:00
BJRambo
f3f40f6859 php7.2호환을 위해 변경한 클래스 Object 를 BaseObject으로 변경 2017-11-28 20:01:28 +09:00
람츠
f4c19a3e0f $request_method 속성에 대한 주석변경
JSON 추가.
2017-11-28 19:41:52 +09:00
람츠
e2704f684a Update Context.class.php 2017-11-28 19:41:41 +09:00
bnu
ec5edff77b 이미 로드한 xe.js 등을 템플릿에서 중복으로 로드하는 코드 제거
d632bffb9f
2017-11-28 19:28:02 +09:00
BJRambo
49f1a691d1
Merge pull request #946 from bjrambo/pr/document-icons
문서 모듈의 아이콘을 스킨으로 설정할 수 있도록 개선
2017-11-28 19:06:21 +09:00
bnu
25b6415587 #2172 복원된 페이지에 남아있는 썸네일 삭제 기능 개선 2017-11-28 18:58:44 +09:00
BJRambo
e3e3a375ec 불필요한 리턴 리디렉트 액션 제거. 2017-11-28 18:57:15 +09:00
BJRambo
7ba2c74e19 사용않함 설정시 실행되는 설명 추가 2017-11-28 18:49:20 +09:00
kijin
9f15787dd2 스킨설정이 없을경우 default 스킨을 가져오도록 추가 2017-11-28 18:42:23 +09:00
BJRambo
75ea89e203 문서 모듈의 아이콘을 스킨으로 설정할 수 있도록 개선
https://github.com/xpressengine/xe-core/pull/2172
2017-11-28 18:07:33 +09:00
Kijin Sung
3eb6e7319f Fix empty agreement becoming required on upgrade
기존에 약관이 없던 사이트도 첫 번째 약관이 필수로 지정되는 버그 수정
@gosugoon
2017-11-28 17:23:09 +09:00
Kijin Sung
1a81fd43f3 Convert Object class to BaseObject in core classes 2017-11-27 22:34:28 +09:00
conory
5bdf95932f kind가 admin이 아닐 경우 (ex. *.controller.php) 확장 모듈로 등록할 수 없는 문제 수정 2017-11-27 21:27:28 +09:00
Kijin Sung
9a83e71bff Allow adding error message and sprintf() variables using setError()
xpressengine/xe-core#2181 적용시 에러 반환 문법을 단순화하기 위한 조치

기존 방식: return new Object(-1, '에러메시지');
XE 제안 방식: return class_exists('BaseObject') ? new BaseObject(-1, '에러메시지') : new Object('에러메시지');
라이믹스 방식: return $this->setError('에러메시지');

기존의 setError() 메소드가 에러 코드만 받을 수 있어서 호환성 보장에 도움이 안 되므로
에러 코드와 에러 메시지를 동시에 넣을 수 있도록 개선하고,
에러 코드를 넣지 않고 에러 메시지만 지정해도 자동으로 -1 에러 코드가 들어가도록 하였음.
(첫 번째 인자가 정수인지 아닌지에 따라 판단함.)

setError(), setMessage(), setMessageType() 등 기존에 무의미한 반환값을 가지던 메소스들 모두
$this를 반환하도록 함으로써 액션이나 트리거 등의 반환값으로 유효하도록 하고,
원할 경우 method chaining까지 사용할 수 있음.

또한 에러메시지에 변수를 넣어야 할 경우
return new Object(-1, sprintf(Context::getLang('error_msg'), $var1, $var2));
이렇게 복잡해지는 문제도 해결하기 위해
setError()에 추가로 넣은 인자는 모두 자동으로 sprintf() 처리를 거치도록 함.
예: return $this->setError('error_msg', $var1, $var2);

즉, 아래와 같은 호출 형태가 모두 유효함.

  - $this->setError(-1);
  - $this->setError(-1, 'error_msg');
  - $this->setError(-1, 'error_msg', $var1, $var2);
  - $this->setError('error_msg');
  - $this->setError('error_msg', $var1, $var2);

단, 이 커밋 이후 신규 작성하는 코어 클래스나 서드파티 자료에서만 사용할 수 있음.
기존 버전과의 호환성을 유지하기를 원하는 서드파티 자료는 XE에서 제안한 삼항식을 사용해야 함.
2017-11-27 16:33:33 +09:00
Kijin Sung
bf6ccfcb44 Force refresh of signup config and signup ruleset if agreement is in old format 2017-11-27 14:52:15 +09:00
Kijin Sung
11c04c5a26 Add multiple member agreements
- 가입 약관, 개인정보처리방침 등을 5개까지 추가할 수 있음
- 약관마다 각각 필수 또는 선택을 지정할 수 있음
- 모든 필수 약관에 동의해야 가입 가능
- 기본 스킨은 수정함. 서드파티 스킨에는 여전히 약관 1개만 표시됨
- 서드파티 스킨 사용시 동의 하나만 선택해도 전체 동의로 간주
2017-11-27 14:43:20 +09:00
Kijin Sung
20f924678a Always enable signup tab in member config screen
회원 가입을 막아 놓으면 회원 설정에서 회원가입 탭아 아예 안 보여서
약관을 수정하거나 확장변수를 추가하는 것도 불가능한 문제를 고침.
2017-11-27 13:06:19 +09:00
Kijin Sung
a8da651f29 Change data type of module_config.config to bigtext
xpressengine/xe-core#2179
xpressengine/xe-core#2180
2017-11-27 13:03:47 +09:00
Kijin Sung
6d6acd5c9e Clean up some comments and line spacing 2017-11-25 19:05:24 +09:00
Kijin Sung
bb51db5956 Add trigger after documentItem->getComments() 2017-11-25 15:51:49 +09:00
Kijin Sung
877863854e Add triggers before and after getCommentList()
예전에 getDocumentList() 전후에 트리거를 추가한 것처럼 댓글 목록에서도
트리거를 사용하여 검색 조건을 바꾸거나 결과를 조정할 수 있도록 함.
2017-11-25 15:32:54 +09:00
Kijin Sung
509b403eff Allow member info to be customized by trigger
문서나 댓글을 작성하거나 수정할 때 트리거에서 회원 정보를 조작할 수
있도록 허용함. $obj->manual_member_info = true 로 바꾸어 반환하면
글쓴이의 정보가 현재 로그인한 사용자나 최초 작성시 정보로 덮어씌워지지
않고 트리거에서 지정한 그대로 남음.

기존에도 $manual_inserted 를 사용하면 이런 기능을 구현할 수 있었으나,
이 변수는 트리거에서 조작할 수 없으며 다른 체크도 모두 건너뛰는 등
여러 가지 불편이 있어서 이렇게 일부 정보만 변경하도록 허용하는 새 방법을
제공함. 익명 게시판 등을 좀더 편리하게 개발할 수 있을 것으로 예상됨.
2017-11-24 23:42:28 +09:00
Kijin Sung
17ca61eed3 Fix #832 make the viewport setting customizable
모바일 접속시 <meta name="viewport"> 태그에 들어가는 내용을
시스템 설정 메뉴에서 커스터마이징할 수 있도록 변경함.
2017-11-23 16:54:06 +09:00
Kijin Sung
2f10d5c0b5 94cdcb2 보완
dispBoardCommentPage 액션은 HTML을 반환하도록 하여 애드온 작동에 지장이 없도록
변경함. (포인트 레벨 아이콘 등이 표시되지 않는 문제가 있었음.)

getBoardCommentPage 액션은 기존 방식처럼 XMLRPC 또는 JSON으로 호출 가능함.
2017-11-23 16:11:51 +09:00
Kijin Sung
460cfd5455 Eliminate unnecessary require_once in mobile classes 2017-11-21 18:09:23 +09:00
Kijin Sung
17c6e48784 Eliminate duplicate methods in communication mobile class 2017-11-21 18:06:04 +09:00
Kijin Sung
94cdcb2e7c Allow dispBoardCategory() and dispBoardCommentPage() to be used in PC
모바일에서만 사용할 수 있었던 dispBoardCategory() 액션을 PC에서도 사용할 수
있도록 변경함. (PC용 스킨에서 category.html 템플릿을 제공하는 경우에 한함)

모바일에서만 사용할 수 있었던 getBoardCommentPage() 액션의 이름을
dispBoardCommentPage()로 바꾸어 PC에서도 사용할 수 있도록 변경하고
기존의 모바일 getBoardCommentPage() 액션은 dispBoardCommentPage()로 연결함.
(get으로 시작하는 액션명은 원래 model에서 써야 하는데 잘못 지정됨)

누가 모바일 스킨을 만들다가 필요해서 추가한 액션인 것 같은데
왜 PC에서는 사용할 수 없도록 해놓았는지 의문임.
비슷한 기능을 PC에서 사용하기 위해 getBoardCommentPageForPC() 등의 액션을
임의로 추가하여 사용하는 사이트가 은근히 많이 보임.
2017-11-21 15:56:17 +09:00
Kijin Sung
9ba0b1e3e6 Add option to control focus on CKEditor
https://www.xetown.com/qna/801549
2017-11-21 15:33:24 +09:00
BJRambo
cb2920bc99
Merge pull request #943 from bjrambo/pr/display-to-send
메세지 보낼때 수신거부인지 확인하여 경고 메세지를 띄움.
2017-11-21 01:05:24 +09:00
BJRambo
e21c203a78 Check if can i receive message when send message. 2017-11-14 06:23:40 +09:00
BJRambo
a23f4f6647 Fix a problem with not running editor on mobile.
Failed to use it in 필요할때 라이믹스 skins.
2017-11-14 06:17:25 +09:00
Kijin Sung
66e8ba4fca Fix URI.js error when processing mailto: links
https://www.xetown.com/rxe_issue/792186
xpressengine/xe-core#2174
2017-11-13 08:44:44 +09:00