Commit graph

364 commits

Author SHA1 Message Date
BJRambo
d2bbe769d4 댓글에 게시글 작성자가 댓글을 남길경우 댓글을 남긴 모든 회원에게 알림 2019-03-13 15:48:45 +09:00
Kijin Sung
0263f63e41 Add trigger before insertNotify so that notifications can be canceled 2019-03-06 01:01:49 +09:00
Kijin Sung
b72c5edcec Include warnings for XE version of advanced mailer and ncenterlite
XE용 고급 메일 발송 모듈이나 알림센터 사용시 표시되어야 하는 오류를
XE용 모듈의 lang.xml에 의존하지 않고 라이믹스에서 직접 제공

cf. https://xetown.com/questions/1179144
2019-02-25 16:27:25 +09:00
BJRambo
db14f7c8c9 알림센터의 알림이 관리자 페이지의 첫페이지에서 노출되는 문제 고침 2019-02-10 15:22:39 +09:00
conory
677e63037f 알림바의 프로필 이미지, 사용자 알림 목록에도 익명처리
afd69cab25 보완
2019-01-24 14:08:08 +09:00
conory
afd69cab25 추천 알림시 추천인 익명 처리를 할 수 있도록 옵션 추가 2019-01-24 13:03:19 +09:00
BJRambo
843e16f983 쪽지함 읽어도 알림이 사라지지 않는 문제 고침 2019-01-17 16:13:01 +09:00
BJRambo
149d819657 영어를 좀 더 명확하게 고침 2018-11-02 14:06:15 +09:00
BJRambo
759f46bbe8 메일알림을 받을시 알림을 알려주는 사이트가 어디인지 명확하게 확인하도록 개선 2018-10-29 13:21:55 +09:00
Kijin Sung
290ee0974d Fix open redirect in ncenterlite module 2018-10-11 09:27:26 +09:00
Kijin Sung
05df47f4e1 Fix #941 use relative target URLS in ncenterlite 2018-10-11 09:08:46 +09:00
Kijin Sung
64de942800 Fix #1099 unnecessary setCookie function definition in ncenterlite skins 2018-10-08 17:55:53 +09:00
Kijin Sung
c49ce899ce Remove duplicate GPL license from ncenterlite 2018-10-02 13:44:46 +09:00
Kijin Sung
6d52a1fbf2 Clean up language in ncenterlite module 2018-10-02 13:44:07 +09:00
BJRambo
2edb8fd2b5 알림이 없을 경우 출력하는 기능 추가.
옵션 제공합니다. 기본값은 사용하지 않도록 되어있습니다.
2018-09-27 19:43:35 +09:00
BJRambo
011918066f 캐시키에 page값을 따로 지정하지 않아 1페이지 이외 캐시 생성 하지 않도록 고침
캐시키값에 page값을 따로지정하지 않고 있음. 따라서 2페이지의 디비가 만약 로딩이 되었더라면 아래의 캐시키를 통해서 2페이지를 로딩할 수 있는 문제점이 있을 수 있음. 따라서 그렇게 되지 않도록 고침
2018-09-27 19:00:11 +09:00
BJRambo
0e8f65fc6c 데이터값도 기본값으로 지정하고 원래대로 순서 변경 2018-09-27 18:38:30 +09:00
BJRambo
c54ef2a08b 라이믹스 코어의 언어 함수를 사용 2018-09-19 17:51:02 +09:00
BJRambo
a94420908d 커스텀 알림을 위한 각 메서드들의 기본값을 제거
커스텀알림을 위한 메서드들에서 대부분 notify_srl 과같은 parameter에 기본값을 사용하고 있습니다.
이 기본값들이 대부분 알림센터의 내부에서 사용되고, 이 커밋에서 지웠던 부분 모두 디비쿼리에 필요한 notify_srl 값을 null으로 전송하여 입력이나 가져오는 것에 대부분 문제가 있었을 것으로 추정하였습니다.
문제가 있을 경우 백지가 나타나게 되어서 문제점을 알 수 없을 수 있는 위험은 있으나 대부분의 동작 코드에서 notify_srl 값이 강제적으로 필요한 것으로 작동되고 있기 때문에 일딴은 parameter의 기본값은 삭제합니다.
2018-09-19 17:50:30 +09:00
BJRambo
58932d22c9 알림을 추가할 때 디비 관련 에러검사 로직을 각각의 처리공간에서 처리 2018-09-19 17:46:42 +09:00
BJRambo
556cc60657 updateFlagFile 메서드의 피라미터위치를 변경합니다.
PHP7.2 이라서 그런지 모르겠지만 일부 두번째 parameter가 requied 일겨우 첫번째 parameter에 default 옵션을 사용할 수 없는 옵션이 생긴 것 같습니다.
이 내용을 그대로 유지할지 바꿀지 고민끝에 모든 서드파티 등등을 검토해보고 사용되는 메서드의 역할에서 알림센터 내부에서만 사용되는 것으로 판단되어 이 메서드의 parameter순서가 변경됩니다.

앞으로 updateFlagFile parameter 을 전달할 경우 순서는 Object, member_srl 입니다.
2018-09-19 17:36:41 +09:00
BJRambo
45892d7419 주석 코드와 사용하지 않는 쿼리 삭제 2018-09-17 20:52:12 +09:00
BJRambo
15ab8b5353 회원 탈퇴시(트리거 발생시) 유저 세팅 데이터도 삭제 할 수 있도록 개선 2018-09-17 19:34:15 +09:00
BJRambo
a20156242f 회원 설정 이용시 리디렉트 방법 개선 2018-09-17 19:29:41 +09:00
Kijin Sung
be1ce69073 Update more modules to throw exceptions 2018-09-05 23:12:08 +09:00
COMI
a32721b6e1 숨겨진 관리자 설정의 설명을 보이도록 변경함 (#1072)
@wincomi 
* Update skin_config.html
* Update module_setup.html
* Update page_info.html
* Update config.html
* Update document_config.html
* Update module_grants.html
* Update skinsetting.html
* Update category_list.html
* Update module_grant_setup.html
2018-08-19 08:51:42 +09:00
Kijin Sung
f12cf74dda Revert "다른 프레임워크와 css 충돌을 피하기 위해 rhymix.less의 class/id 이름에 prefix 처리"
This reverts commit b1cb1f3f34.
2018-08-04 20:56:24 +09:00
conory
b1cb1f3f34 다른 프레임워크와 css 충돌을 피하기 위해 rhymix.less의 class/id 이름에 prefix 처리
변경전의 xe.less 파일도 rhymix.less와 함께 load 하여 호환정 유지

CSS 충돌 예)
bootstrap의 .btn
사용 예)
bootstrap 사용시엔 {Context::unloadBasicFiles('xe')}으로 unload 시키면 문제없음
2018-08-03 19:45:30 +09:00
Kijin Sung
802ca2cf1b Fix incorrect sender info in new message notification
A 회원 계정으로 로그인한 상태에서 애드온이나 서드파티 모듈이
B 회원으로부터 C 회원에게 쪽지가 발송되도록 하면,
C회원에게 표시되는 쪽지 알림에는 A 회원으로부터 쪽지가 왔다고
잘못 표시되는 문제 수정.

- 쪽지 발송 트리거에서 sender_srl을 참조하는 코드가 누락되어 있었음.

- insertNotify에서 명시적으로 지정한 target_member_srl 속성보다
  현재 로그인한 사용자의 회원정보를 우선 처리하는 문제가 있었음.
  이러면 target_member_srl 속성을 쓰는 이유가 없음 ㅡ.ㅡ;;
2018-01-25 23:39:03 +09:00
Kijin Sung
c9d3645f10 Automatically fill cache in ncenterliteModel->_getNewCount()
285ea8a 보완

카운트 캐시가 없는 경우 자동으로 캐시를 생성하도록 변경함.
푸시앱 등 알림 내용은 안 불러오고 카운트만 받아가는 경우가 있어서
이렇게 하지 않으면 반복 호출해도 캐시 효과가 없음.

푸시앱 사용자가 많고 알림이 수천~수만 개씩 쌓이는 사이트에서
이 패치 적용 후 DB서버 부하가 50% 이상 줄어들었음.
2018-01-16 23:36:37 +09:00
Kijin Sung
285ea8a8ed Apply object cache to ncenterliteModel->_getNewCount()
알림이 많이 쌓여 있는 경우 푸시앱 등 알림센터에 직접 접근하는 서드파티 자료에서
알림 수를 가져올 때 부하가 발생함. 이미 알림 목록을 캐싱하도록 되어 있으므로
캐시를 활용하여 불필요한 SELECT COUNT(*) 쿼리를 하지 않도록 개선함.
2018-01-16 23:16:27 +09:00
BJRambo
564944263b 값을 조건문밖으로 빼서 아래 변수사용할 수 있도록 고침 2018-01-15 15:51:45 +09:00
BJRambo
5ceac274e4 702d94a 보안 페이지값이 1이상일 경우 디비에서 페이지리스트를 가져와야함 2018-01-15 15:41:04 +09:00
Kijin Sung
702d94a9e7 Use object cache to store notification list in ncenterlite
오브젝트 캐시 사용시 알림센터에서 플래그 파일 대신 캐시를 활용하도록 변경.
기존 방식에서 오브젝트 캐시로 전환시 플래그 파일은 삭제됨.
2018-01-15 14:20:51 +09:00
Kijin Sung
25369a15f8 Eliminate slow query from ncenterlite auto read trigger
글읽기시 해당 글과 관련된 알림을 자동으로 읽기 처리해 주는 루틴에서
불필요한 SELECT COUNT(*) 쿼리와 플래그 파일 갱신이 발생하는 문제 수정.

SELECT COUNT(*) 쿼리 없이 무조건 업데이트부터 하고
affected rows 값에 따라 플래그 파일 갱신 여부를 결정하도록 함.
2018-01-15 14:01:16 +09:00
BJRambo
b6dd614dc7 관리자의 경우 회원 닉네임이 노출되지 않는 문제 고침 2018-01-11 20:35:19 +09:00
BJRambo
8f2f4aae42 Fix #968 알림센터의 유저 알림기본 동작을 항상 받는 동작으로 개선 2018-01-11 20:31:08 +09:00
Kijin Sung
f41c976fcf Add index hint to ncenterlite.getNotifyNewCount 2018-01-04 14:01:43 +09:00
Kijin Sung
d6428bb71d Add index hint to ncenterlite.getNotifyList query 2018-01-04 12:08:05 +09:00
Kijin Sung
28a2398e91 Fix null value in some circumstances in insertNotify() 2017-12-21 00:27:02 +09:00
Kijin Sung
b9074335aa Fix inconsistent document & comment vote handling in ncenterlite 2017-12-21 00:20:14 +09:00
Kijin Sung
f0ad692650 Enable responsive skin in member, communication, and ncenterlite
회원, 커뮤니케이션, 알림센터Lite 모듈에서 모바일 스킨을 선택할 때
PC와 동일한 반응형 스킨을 지정할 수 있도록 개선
2017-12-05 23:58:04 +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
BJRambo
f3f40f6859 php7.2호환을 위해 변경한 클래스 Object 를 BaseObject으로 변경 2017-11-28 20:01:28 +09:00
Kijin Sung
460cfd5455 Eliminate unnecessary require_once in mobile classes 2017-11-21 18:09:23 +09:00
BJRambo
2c3de50678 Fix #940 member_srl 과 로그인정보가 없을 경우에 리턴하도록 고침. 2017-11-05 21:33:33 +09:00
BJRambo
60a08982f5 Fix typo. 2017-11-05 14:15:25 +09:00
BJRambo
7f42031e2a Fix #940 fix the ability to pass logged_info when member_srl is empty. 2017-11-05 14:14:01 +09:00
BJRambo
ad2beed710 Fix display notify menu depends on user notify setting 2017-08-19 17:18:03 +09:00
BJRambo
063e22152d Fix typo to newline in en.php. 2017-08-19 17:07:07 +09:00