Skip to content
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

확장변수 검색 시 상담 기능이 작동하지 않는 버그 수정 #463

Merged
merged 3 commits into from
Feb 24, 2014
Merged

확장변수 검색 시 상담 기능이 작동하지 않는 버그 수정 #463

merged 3 commits into from
Feb 24, 2014

Conversation

smaker
Copy link
Contributor

@smaker smaker commented Feb 11, 2014

확장 변수 순으로 정렬 시에는 상담 기능이 제대로 작동하지만, 확장 변수로 검색을 했을 때는 상담 기능이 작동하지 않아서 수정해보았습니다.

document.getDocumentListWithinExtraVars
document.getDocumentListWithExtraVarsPage

이 두 쿼리에 member_srl 을 처리하는 부분이 빠져있더군요.

@ngleader
Copy link
Contributor

+1

<condition operation="equal" column="extra_vars.module_srl" default="documents.module_srl" pipe="and" />
<condition operation="equal" column="extra_vars.document_srl" default="documents.document_srl" pipe="and" />
<condition operation="equal" column="extra_vars.var_idx" var="var_idx" notnull="notnull" pipe="and" />
<condition operation="in" column="documents.status" var="statusList" pipe="and" />
<condition operation="like" column="extra_vars.value" var="var_value" notnull="notnull" pipe="and" />

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@smaker 공칸 제거해 주세요.

@smaker
Copy link
Contributor Author

smaker commented Feb 21, 2014

@akasima 수정했습니다 :)

akasima added a commit that referenced this pull request Feb 24, 2014
확장변수 검색 시 상담 기능이 작동하지 않는 버그 수정
@akasima akasima merged commit 6f58ece into xpressengine:develop Feb 24, 2014
@akasima akasima added this to the 1.7.5 milestone Feb 24, 2014
@ghost ghost assigned akasima Feb 26, 2014
@smaker smaker deleted the develop.extra_vars branch June 21, 2014 01:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants