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

[버그수정] 910. 당직관리 > 당직엑셀등록 오류 수정 #500

Open
wants to merge 1 commit into
base: contribution
Choose a base branch
from

Conversation

taeseong
Copy link

@taeseong taeseong commented Sep 23, 2024

수정 사유 Reason for modification

소스를 수정한 사유가 무엇인지 체크해 주세요. Please check the reason you modified the source. ([X] X는 대문자여야 합니다.)

  • 버그수정 Bug fixes
  • 기능개선 Enhancements
  • 기능추가 Adding features
  • 기타 Others

수정된 소스 내용 Modified source

수정된 파일

  • EgovBndtManage_SQL_altibase.xml
  • EgovBndtManage_SQL_cubrid.xml
  • EgovBndtManage_SQL_goldilocks.xml
  • EgovBndtManage_SQL_maria.xml
  • EgovBndtManage_SQL_mysql.xml
  • EgovBndtManage_SQL_oracle.xml
  • EgovBndtManage_SQL_postgres.xml
  • EgovBndtManage_SQL_tibero.xml
  • EgovBndtManageBndeListPop.jsp

수정 내용

  • EgovBndtManage_SQL
    • 당직엑셀 등록 시 소속 정보를 불러오지 못하는 문제 수정
    • 엑셀 등록 시 MyBatis 1차 캐시로인해 값이 올바르게 등록되지 않는 문제 수정

AS-IS

<select id="selectBndtManageBnde" resultType="egovframework.com.uss.ion.bnt.service.BndtManageVO">
select A.ESNTL_ID as bndtId ,
    A.USER_NM as tempBndtNm ,
    (select info.ORGNZT_NM from COMTNORGNZTINFO info where  info.ORGNZT_ID = A.ORGNZT_ID )  AS tempOrgnztNm,
    (select count(user_nm) from  COMVNUSERMASTER where user_nm = A.USER_NM) tempCount
from COMVNUSERMASTER A
WHERE  USER_ID = #{tempBndtId} 
AND    USER_NM   = #{tempBndtNm}
limit 1
</select>

TO-BE

<!-- 같은 사용자 정보를 조회 시 MyBatis 1차 캐시로 인한 객체 캐싱 방지를 위해 flushCache="true" 추가 -->
<select id="selectBndtManageBnde" resultType="egovframework.com.uss.ion.bnt.service.BndtManageVO" flushCache="true">
select A.ESNTL_ID as bndtId ,
    A.USER_NM as tempBndtNm ,
    (select info.ORGNZT_NM from COMTNORGNZTINFO info where  info.ORGNZT_ID = A.ORGNZT_ID )  AS tempOrgnztNm,
    (select count(user_nm) from  COMVNUSERMASTER where user_nm = A.USER_NM) tempCount
from COMVNUSERMASTER A
WHERE  ESNTL_ID   = #{tempBndtId} <!-- 엑셀에서는 ESNTL_ID을 입력하는데 쿼리는 USER_ID로 되어있어 ESNTL_ID로 수정 -->
AND    USER_NM   = #{tempBndtNm}
limit 1
</select>
  • EgovBndtManageBndeListPop.jsp
    • 엑셀 업로드 후 결과 값 변수명을 엑셀 데이터에 맞게 수정

AS-IS

<input type="hidden" name="bndtId" id="bndtId" value="${resultInfo.tempBndtId}">

TO-BE

<input type="hidden" name="bndtId" id="bndtId" value="${resultInfo.bndtId}">

JUnit 테스트 JUnit tests

테스트를 완료하셨으면 다음 항목에 [대문자X]로 표시해 주세요. When you're done testing, check the following items.

  • JUnit 테스트 JUnit tests
  • 수동 테스트 Manual testing

테스트 브라우저 Test Browser

테스트를 진행한 브라우저를 선택해 주세요. Please select the browser(s) you ran the test on. (다중 선택 가능 you can select multiple) [X] X는 대문자여야 합니다.

  • Chrome
  • Firefox
  • Edge
  • Safari
  • Opera
  • Internet Explorer
  • 기타 Others

테스트 스크린샷 또는 캡처 영상 Test screenshots or captured video

수정 전 - 엑셀 업로드 후 소속 값 없음

image

수정 후 - 엑셀 업로드 후 소속 값 표시됨 (조직이 없는 사용자는 미표시됨)

image

업로드 실행 결과

2024-09-23.8.45.25.mov

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant