New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

게시물, 댓글 고유 링크에 mid를 포함하도록 수정 #2168

Merged
merged 1 commit into from Nov 24, 2017

Conversation

3 participants
@smaker
Contributor

smaker commented Oct 18, 2017

#2155 이슈를 수정한 PR입니다.

Content 위젯과 RSS 모듈에서 mid를 포함한 링크가 걸려있는 지 확인해보았습니다.
빠진 곳 있으면 알려주세요.

*/
function getCommentMid()
{
$model = getModel('module');

This comment has been minimized.

@bjrambo

bjrambo Oct 18, 2017

Contributor

$model 이렇게 적으면 어떤 모델을 불러오는지 명확한 네이밍이 아닌것 같습니다.

정확하게 $oModuleModel = (ObjectModuleModel) 으로 명확한 이름이 나오도록 하는것이 좋겠습니다.

아래의 $module 이라는 이름도 역시 마찬가지로 $moduleInfo 혹은 $module_info 으로 명확하게...

@bjrambo

bjrambo Oct 18, 2017

Contributor

$model 이렇게 적으면 어떤 모델을 불러오는지 명확한 네이밍이 아닌것 같습니다.

정확하게 $oModuleModel = (ObjectModuleModel) 으로 명확한 이름이 나오도록 하는것이 좋겠습니다.

아래의 $module 이라는 이름도 역시 마찬가지로 $moduleInfo 혹은 $module_info 으로 명확하게...

This comment has been minimized.

@smaker

smaker Nov 2, 2017

Contributor

지적 감사합니다. 알려주신대로 수정해서 놓겠습니다.

@smaker

smaker Nov 2, 2017

Contributor

지적 감사합니다. 알려주신대로 수정해서 놓겠습니다.

@bnu bnu self-assigned this Oct 24, 2017

@bnu bnu added this to the 1.8-2017-51W milestone Oct 27, 2017

@bnu bnu added this to 처리 중 in 이슈 진행 상황 Nov 23, 2017

@bnu bnu merged commit 108408c into xpressengine:develop Nov 24, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@bnu bnu removed this from 해결 중 in 이슈 진행 상황 Nov 24, 2017

@bnu bnu modified the milestones: 1.8-2017-51W, 1.8.47, 1.9.0 Nov 24, 2017

bnu added a commit that referenced this pull request Nov 28, 2017

#2155 게시물 주소에 생략된 mid 추가
- #2168 PR과 동일하게 mconent 위젯 수정
  - @smaker 님 코드 참고

@bnu bnu added this to 이슈 종료 in 이슈 진행 상황 Nov 30, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment