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

fix #1677 MSSQL 쿼리 구문 오류 해결 #1679

Merged
merged 1 commit into from Nov 28, 2017

Conversation

2 participants
@RapidWorkers
Contributor

RapidWorkers commented Jul 27, 2015

github 어렵습니다 ㅠㅠ
#1677 MSSQL 쿼리 구문 오류 해결

수정내용 : 일부 모듈 등에서 예약어를 칼럼명으로 사용함에 따라 MSSQL에서 인식하지 못하고 뱉어내는 오류를 해결하기 위해 큰따옴표를 추가하였습니다. 이미 배포된 모듈이 많고 향후에도 이런 일이 발생할 수 있기에 PR 올립니다.

SQL Server Express 10.50.2789.0.v1, PHP 5.4.24 환경에서 테스트 되었습니다.

@RapidWorkers RapidWorkers changed the title from #1677 MSSQL 쿼리 구문 오류 해결 to fix #1677 MSSQL 쿼리 구문 오류 해결 Jul 28, 2015

@bnu bnu added the status/keep label Feb 1, 2017

@bnu bnu closed this Feb 1, 2017

@bnu bnu added this to 검토 중 in 이슈 진행 상황 Nov 28, 2017

@bnu bnu self-assigned this Nov 28, 2017

@bnu bnu added this to the 1.9.0 milestone Nov 28, 2017

@bnu bnu moved this from 검토 중 to 해결 예정 in 이슈 진행 상황 Nov 28, 2017

@bnu bnu reopened this Nov 28, 2017

@bnu bnu merged commit 92b4c32 into xpressengine:develop Nov 28, 2017

1 check passed

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

이슈 진행 상황 automation moved this from 해결 예정 to 이슈 종료 Nov 28, 2017

@RapidWorkers RapidWorkers deleted the RapidWorkers:patch-1 branch Dec 19, 2017

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