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

사이트 메뉴 편집중 권한 선택그룹 저장 불가 문제 #2193

Closed
itsurea opened this issue Oct 3, 2023 · 8 comments
Closed

사이트 메뉴 편집중 권한 선택그룹 저장 불가 문제 #2193

itsurea opened this issue Oct 3, 2023 · 8 comments
Labels
bug 버그로 확인된 사항 patch available 패치 있음 표시 (bug와 함께 사용)

Comments

@itsurea
Copy link

itsurea commented Oct 3, 2023

image

선택그룹으로 요청시 exposure 값이 빈 상태로 요청이 들어가서 선택그룹으로 권한 설정이 불가능함

@kijin
Copy link
Member

kijin commented Oct 3, 2023

정상적으로 설정했을 때 제보하신 증상이 확인되지 않습니다.

"선택 그룹 사용자"를 선택한 후, 아래에서 아무 그룹도 선택하지 않으면 exposure 값이 빈 값으로 들어갈 수는 있습니다.

@kijin kijin added the unconfirmed 문제 재현 가능 여부가 확인되지 않음 / 제보자 미협조 label Oct 3, 2023
@itsurea
Copy link
Author

itsurea commented Oct 3, 2023

선택 그룹 사용자 선택이후에 js 쪽을 봤는대 checked를 읽어오지 못해서 체크가 안되는거 같은데...
따로 js 에러가 뜨지도 않는데 어떻게 디버깅 해봐야할까요?

@kijin
Copy link
Member

kijin commented Oct 4, 2023

혹시 이 부분의 checked 말씀이신가요?

https://github.com/rhymix/rhymix/blob/master/modules/menu/tpl/sitemap.html#L2443

메뉴 편집 기능은 낡은 코드가 굉장히 복잡하게 얽혀 있어서, 정확히 어떤 경로로 문제의 기능에 들어가서 어느 부분을 확인하셨는지 자세히 알려주시면 큰 도움이 됩니다. 메뉴의 종류, 몇 단계로 깊이 들어갔는지, 어떤 그룹을 선택하셨는지, 혹시 고급 설정에서 jQuery 버전을 변경하셨는지 등등...

만약 위에 링크해 드린 부분의 의심되신다면 attr('checked')prop('checked')is(':checked')로 변경하고 테스트해 보시기 바랍니다.

@itsurea
Copy link
Author

itsurea commented Oct 4, 2023

props('checked')는 작동 하였습니다. jquery 버전은 따로 변경하지 않았고 1차메뉴에서 해당 문제가 발생하고 있었습니다.

@kijin
Copy link
Member

kijin commented Oct 4, 2023

브라우저 버전이나 다른 자료의 유무에 따라 동작이 달라지는 것 같습니다만, 아무튼 체크박스 체크 여부를 확인하는 데 attr를 사용하는 것은 예전 방식이므로 prop으로 변경해 보았습니다. 위에 링크된 d84e0e9 패치 적용시 정상 동작하는지 확인해 주실 수 있으면 감사하겠습니다.

@itsurea
Copy link
Author

itsurea commented Oct 4, 2023

패치적용이후 정상 작동됩니다 감사합니다.

@kijin kijin added bug 버그로 확인된 사항 patch available 패치 있음 표시 (bug와 함께 사용) and removed unconfirmed 문제 재현 가능 여부가 확인되지 않음 / 제보자 미협조 labels Oct 4, 2023
@kijin
Copy link
Member

kijin commented Oct 4, 2023

확인 감사합니다.^^

@kijin kijin closed this as completed in d84e0e9 Oct 10, 2023
@kijin
Copy link
Member

kijin commented Oct 10, 2023

아하, 다른 이슈를 보니 jQuery 3을 사용하고 계셨군요.

사이트 메뉴 편집, 위젯페이지 편집 등 오래 전에 만들어진 기능을 계속 쓰고 있는 부분에서는 jQuery 3에서 오류가 발생할 수 있습니다. 프론트엔드 문제를 제보하실 때 jQuery 3이라고 미리 말씀해 주시면 문제 확인 및 디버깅에 도움이 되겠습니다.

감사합니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 버그로 확인된 사항 patch available 패치 있음 표시 (bug와 함께 사용)
Projects
None yet
Development

No branches or pull requests

2 participants