Home

Welcome to the 137A wiki!

주제 : xpressengine xe-core

(1) 개발을 위해서 사용된 언어인 PHP에 대해서 학습해본다.

-간단한 coding을 시도해본다.

(2) 개발 Github 사이트를 찾아본다.

-공부해본 PHP에 대해서 코드의 틀린 부분을 수정 시도 해본다. 하지만 불가능하다면, 코드의 오탈자에 대해서 수정해본다.

-가능하면 직접 xpressengine을 사용해보고 불편한 점이나 좋은 점에 대해서 알아본다.

-메뉴얼 등의 파일에서 오탈자를 수정해본다.

-코드의 오류 부분에 대해서 pull-request를 보내본다.(메뉴얼에서 요구하던 부분에 대한)

팀원 별 역할과 활동

-은수연 : Fork, Pull-Request, 코드 수정, 활동 내역 정리

-양혜리 : xpressengine 직접 사용해보고 불편한 점 알아보기, 코드 수정

-강평 : xpressengine 직접 사용해보고 불편한 점 알아보기, 코드 수정

수행한 부분

1. PHP에 대해서 기초를 공부해 본 뒤 PHP로 helloworld 출력하였다. (code 부분 참조)

2. xpressengine 체험 설치 방법에 대해서 알아보고 업로드하였다.

3. 메뉴얼 부분이나 소개 부분 등의 파일에서 띄어쓰기에 대한 부분을 수정하였다.

4. 한글의 오타에 대한 부분을 수정하였다.

5. 메뉴얼에서 요구하는 부분인 if, for문 뒤의 {}는 아랫 줄에 내려서 쓰는 것이었는데 그렇게 작성하지 않은 코드가 많아서 수정을 하였다.

6. 코드 안에서 PHP 파일에 대한 버전 정보가 잘못되어 있어서 수정하였다.

7. 틀린 부분이 많은 곳의 코드 정리를 해서 Pull-Request를 하였다.

다음은 Pull-Request에 대한 회신입니다.

첫 번째 회신

•Pull request는 develop 브랜치로 넣어주세요.

•엉뚱한 위치에 새 파일을 만들지 말고 기존 파일을 그대로 수정한 후 커밋하셔야 어디가 어떻게 바뀌었는지 확인하기 쉽습니다.

•첨부하신 파일에도 } 뒤에 바로 else가 나오는 등, 일관성없는 코드가 많이 남아 있네요.

•그러나 자바스크립트는 PHP와 달리 콜백 함수가 많고, 줄바꿈시 세미콜론 자동 삽입 등의 문제가 생길 수 있으므로 PHP 쪽에 적용되는 코딩 규칙을 모두 지키지는 않는 것이 좋습니다.

두 번째 회신

규칙이 많이 깨지긴 했습니다.

하지만, 문법 정리는 되도록이면 하지 않으려고 합니다.

코드를 고쳐서 별도의 저장소로 관리하며 merge하는 경우 잦은 충돌이 예상되기 때문입니다.

문법을 정리해주는 툴도 있어서 전체 정리가 가능하지만, 코드가 고쳐지는 부분이 아니면 유지하고 있습니다.