rhymix icon indicating copy to clipboard operation
rhymix copied to clipboard

익명 게시판으로 전환 후 기존 작성된 글 수정 시 닉네임만 익명이 되는 문제

Open dewekk opened this issue 3 years ago • 2 comments

문제 상황

익명 게시판 전환 후 기존 작성된 글을 수정 시 닉네임 표시만 익명으로 변경됩니다.

문서

  1. 게시판 익명 (X) -> A글 작성
  2. 게시판 익명 (O) -> A글 수정
  3. A글 member_srl 양수 닉네임 익명

댓글

  1. 게시판 익명 (X) -> B댓글 작성
  2. 게시판 익명 (O) -> B댓글 수정
  3. B댓글 1번상태 유지

원인

문서와 댓글 익명 처리 방식이 아래와 같이 다릅니다.

문서

  1. board 모듈에서 https://github.com/rhymix/rhymix/blob/aa78a03b99df9c7deabfa4fb2d498eed067f54c8/modules/board/board.controller.php#L103
  • [닉네임 익명]은 모든 상황에서 처리
  • [member_srl 음수 처리]는 insert상황에서만 하고 update상황에서는 하지 않음. (temp제외)
  1. document 모듈에서
  • [insert] $manual_inserted, [update] $manual_updated로 board에서 넘겨준 member_info가 유지 됨. https://github.com/rhymix/rhymix/blob/aa78a03b99df9c7deabfa4fb2d498eed067f54c8/modules/document/document.controller.php#L908

댓글

  1. board 모듈에서 https://github.com/rhymix/rhymix/blob/aa78a03b99df9c7deabfa4fb2d498eed067f54c8/modules/board/board.controller.php#L457
  • [닉네임 익명]은 모든 상황에서 처리
  • [member_srl 음수 처리]는 insert, update 모두 함
  1. comment 모듈에서
  • [insert] $manual_inserted로 board에서 넘겨준 member_info가 유지 됨.
  • [update] $manual_updated가 체크 되지 않아 member_info가 logged_info로 대체 됨 https://github.com/rhymix/rhymix/blob/aa78a03b99df9c7deabfa4fb2d498eed067f54c8/modules/comment/comment.controller.php#L849

쟁점사항

  1. 익명 게시판으로 전환되었을 때 게시물 및 댓글 수정 시 익명으로 전환되는 것이 맞는지 현상태 유지가 맞는지
  2. board 모듈에서 insert 상황에서만 익명 처리 할지 comment모듈 처럼 document모듈에서 update시 logged_info로 대체 할지

어느 한쪽으로 문서, 댓글 결과가 통일 되어야 할 것 같아 이슈 남깁니다.

dewekk avatar Sep 02 '22 04:09 dewekk

글이든 댓글이든 작성 당시에 입력한 (or 자동 입력된) 닉네임이 그대로 유지되는 것이 맞다고 봅니다. 굳이 nick_name 컬럼을 업데이트할 필요가 없다면 해당 변수를 쿼리에 넘기지 않는 것이 이상적이겠네요.

kijin avatar Oct 10 '22 16:10 kijin

document모듈과 comment모듈의 로그인 정보로 대체 하는 조건문이 다른 부분은 호환성을 고려하여 그대로 유지하고, board모듈에서 익명 정보 생성을 글 작성 시와 임시 문서 작성 시에만 하여 닉네임을 업데이트 하지 않도록 변경하는 PR 하였습니다.

dewekk avatar Oct 11 '22 08:10 dewekk