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

get_board_db Hook이 제대로 동작하지 않는 문제 수정 #298

Closed
wants to merge 2 commits into from

Conversation

kkigomi
Copy link
Contributor

@kkigomi kkigomi commented Dec 17, 2023

get_board_db() 함수에서 get_board_db Hook이 실행될 때 첫번째 콜백에 항상 빈 배열이 전달되어 Hook이 제대로 동작할 수 없고, 이 get_board_db() 함수 내부에서 정적변수로 캐시한 데이터 마저 역시 빈 배열로 대체시켜 cache 도 제대로 동작하지 않는 문제를 수정합니다.

현재는 아래와 같은 문제로 인해 Hook의 활용과 Cache가 제대로 동작하지 않아 DB 조회를 줄이려는 의도가 동작하지 않습니다.

  • 콜백에 항상 빈 배열이 전달 됨
  • 잘못 처리된 Hook 사용으로 get_board_db() 함수 내의 $cache[$key] 값을 항상 빈배열로 대체되어 cache가 활용되지 않음

- 콜백에 항상 빈 배열이 전달되는 문제 수정
- `get_board_db()` 함수 내의 `$cache[$key]` 값을 항상 빈배열로 대체시키는 문제 수정
@kkigomi kkigomi marked this pull request as draft April 25, 2024 09:17
@kkigomi kkigomi closed this May 7, 2024
@kkigomi kkigomi deleted the feature/get_board_db branch May 7, 2024 06:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants