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

일부 서드파티에서 _XE_LOCATION_ 을 사용하고 있습니다. #2160

Closed
bjrambo opened this issue Aug 4, 2023 · 2 comments
Closed
Labels
bug 버그로 확인된 사항 compatibility 호환성 고려 필요 patch available 패치 있음 표시 (bug와 함께 사용)

Comments

@bjrambo
Copy link
Member

bjrambo commented Aug 4, 2023

일부 서드파티에서 언어의 상수인 _XE_LOCATION_ 을 호출하는 것을 확인했습니다..

XE때 있던 내용으로 안에 내용이 'ko'이라고 적혀있는것을 확인된바 아마 언어관련 내용이 아닌가 싶네요..

https://github.com/xpressengine/xe-core/blob/develop/config/package.inc.php

해당 파일에 선언되어있던 친구엿던것 같은데.. 이게 PHP8.0 이상 넘어오면서 일부 서드파티에서 해당 상수를 호출하게되면 에러를 출력하는게 문제가 되엇습니다.

@YJSoft
Copy link
Contributor

YJSoft commented Aug 4, 2023

https://github.com/xpressengine/xe-core/blob/abde50d61df41e7282a01da7570ead694d957263/modules/admin/admin.admin.view.php#L210-L212

XE 글로벌화 시도의 흔적입니다. 언어에 따라 한국은 xpressengine.com, 이외 xpressengine.org로 이동시키려는 시도였죠.
애초에 서드파티에서 쓰라고 만든 상수가 아니므로(package.inc.php 파일에서 ko로 고정하고 있으므로 아무 의미 없습니다) 해당 상수를 참조하고 있다면 잘못 사용하고 있는 것입니다만... 하위 호환성을 위해 선언정도는 해줘야 할것 같네요.

@bjrambo
Copy link
Member Author

bjrambo commented Aug 4, 2023

확인해보니 첫링크 admin.admin.view.php 파일에 선언된 내용과 거의비슷하게 구현을 하는 경우가 많았던 것 같아요.

기존에는 거의 복붙식으로 구현해두는 자료들이 많아서 옛날 자료들이 많이 그랫던 것 같아요.

현재 누리고 모듈에서 우선 발견을 하긴햇는데... 그 외 모듈들에서 몇개는 사용중인 모듈들이 있을 것 같아요

@kijin kijin added bug 버그로 확인된 사항 compatibility 호환성 고려 필요 patch available 패치 있음 표시 (bug와 함께 사용) labels Aug 4, 2023
@kijin kijin closed this as completed in f1a898e Aug 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 버그로 확인된 사항 compatibility 호환성 고려 필요 patch available 패치 있음 표시 (bug와 함께 사용)
Projects
None yet
Development

No branches or pull requests

3 participants