Ji Yong, Kim

Results 20 issues of Ji Yong, Kim

procBoardAdminUpdateBoard**Fro**Basic 라고 되어있네요. +. 이런 단순 오타와 같은 이슈들을 한데 모아 정리할 수 있는 곳이 있었으면 합니다. 크게 의미있는 이슈도 아니고, 작동에 문제가 있는 이슈도 아닌데, 그래도 언젠가 한번쯤 수정은...

cleanup
compatibility

현재 라이믹스는 triggerCall 메서드에서 $type에 '\\' 가 포함되면, 클래스명을 `$class_name = sprintf('Rhymix\\Modules\\%s\\%s', $module, $type);` 와 같은 방식으로 조합해 처리하고 있습니다. --- 문제는... `Rhymix\Modules\Foo\Triggers` 클래스에 트리거를 담아놓게 되면 등록시 저 클래스를...

discussion

1. default 속성 ```xml base_container ``` 많은 레이아웃, 위젯, 애드온 등에서 extra_vars에 대한 기본값을 필요로 합니다. 위 코드와 같이 default 속성을 제공하면 특히 스킨에서 php 코드를 사용하는 양이 확 줄것...

feature request

모듈이나 애드온 설치 시 Github 에서 자료를 다운로드 받아 그대로 업로드 하는 경우가 많습니다. 특히 Github에 익숙하지 않은 유저분들, 또 특히 SSH 를 통해 설치하지 않고 압축파일을 다운로드 받아 업로드...

feature request

라이믹스 2.0에 들어서 /module/action 형태의 짧은 주소를 지원하기 시작했습니다. 관리자 액션도 동일하게 적용되는데요. (/admin/dispMemberAdminList) 관리자 페이지의 좌측 메뉴에서는 짧은 주소가 적용되지 않은 링크로 이동됩니다. 의도된 작동은 아닌것으로 보입니다.

wontfix

https://github.com/rhymix/rhymix/blob/aa78a03b99df9c7deabfa4fb2d498eed067f54c8/modules/document/document.controller.php#L1781-L1809 추천 실패 시 트랜잭션을 롤백하지 않고 바로 리턴을 던지고 있는데요. 코어만 따지고 보면 큰 문제는 없을것으로 보이나, 서드파티가 끼어들면 잘못된 트랜잭션을 커밋하거나 롤백하는 문제가 발생할 수 있을 듯 합니다....

bug
patch available

https://github.com/rhymix/rhymix/blob/aa78a03b99df9c7deabfa4fb2d498eed067f54c8/modules/member/tpl/member_list.html#L123-L149 124, 140번 라인의 {@$isGoTo = true} 127, 143번 라인의 cond="$isGoTo" 전혀 의미없는 코드 같습니다. 또, 127, 143에서 동일한 아이디 goTo 가 중복 선언되어 정상 작동하는지도 의문입니다.

cleanup

CDN에서 폰트를 끌어와 사용할 경우, font-display: swap이 적용되지 않아 아쉬움이 많습니다. 해당 속성이 추가된 CSS 파일도 같이 제공된다면 정말 좋을 것 같습니다!

https://github.com/editor-js/editorjs-php/blob/9446738b6046ab05fe850dec2814dec535fe2e48/README.md?plain=1#L187-L207 I think it should be... ```json { "blocks": [ { "type": "list", "data": { "items": [ "first", "second", "third" ], "style": { "background-color": "red", "font-color": "black" } } }...

코드는 전부 이슈 작성하면서 그때 그때 생각나는 방식을 작성한거라, 논의가 필요합니다. --- 1. 이후 세그먼트를 모두 무시하는 지시자 ```xml ``` https://example.com/mid/foo https://example.com/mid/foo/bar/baz 등 모든 `foo` 이후로 오는 세그먼트는 무시하고 라우팅하는...

improvement