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

[버그수정] 780. 통합링크관리 > Uncaught TypeError 오류 수정 #502

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

Conversation

taeseong
Copy link

수정 사유 Reason for modification

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

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

수정된 소스 내용 Modified source

수정된 파일

  • EgovUnityLinkUpdt.jsp

수정 내용

  • fn_egov_save_UnityLink 함수의 varFrom.onsubmit(); 코드에서 Uncaught TypeError 에러 발생
  • 함수 내에서 varFrom.onsubmit(); 코드가 의미가 없어 제거하였습니다.

AS-IS

function fn_egov_save_UnityLink(){
	var varFrom = document.unityLink;
	varFrom.onsubmit(); // 해당 코드 제거
	if(confirm("<spring:message code="common.save.msg" />")){
		varFrom.action =  "<c:url value='/uss/ion/ulm/updtUnityLink.do' />";
		if(!validateUnityLink(varFrom)){
			return;
		}else{
			varFrom.submit();
		}
	}
}

TO-BE

function fn_egov_save_UnityLink(){
	var varFrom = document.unityLink;
	if(confirm("<spring:message code="common.save.msg" />")){
		varFrom.action =  "<c:url value='/uss/ion/ulm/updtUnityLink.do' />";
		if(!validateUnityLink(varFrom)){
			return;
		}else{
			varFrom.submit();
		}
	}
}

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

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