Distributor: NetworkSiteConnection::update_syndicated가 Gutenberg/REST 컨텍스트에서 너무 일찍 실행됨

@lakrisgubben 매우 상세한 문제와 이에 대한 연구에 감사드립니다. 대단히 감사합니다! 이 문제를 해결할 수 있는지 확인하기 위해 다음 이정표 릴리스로 가져오고 있지만 PR을 할 수 있다면 검토를 통해 다음 릴리스로 기꺼이 가져갈 것입니다. 다시 한번 감사합니다!

@jeffpaul 답변 감사합니다! 이에 대한 PR을 도와드리게 되어 기쁩니다. 하지만 그렇게 하기 위해 코드베이스에 정통한 사람이 여러분 모두가 만족할 수 있는 방식으로 수정되었는지 확인하고 싶습니다. :)

@lakrisgubben 이 문제를

솔직히, 우리는 대부분 구텐베르크의 작동 방식에 의존하기 때문에 여기에 훌륭한 접근 방식이 있는지 확신할 수 없습니다. 이것은 알려진 문제(여기에 대한 한 스레드 참조: https://github.com/WordPress/gutenberg/issues/12903)이며 유사한 시나리오에 대한 좋은 접근 방식을 파악하려는 것처럼 들립니다.

다음은 귀하의 접근 방식과 매우 유사한 제안입니다.

• 특정 게시물이 Gutenberg를 사용하여 작성되었는지 여부를 알려주는 is_using_gutenberg 도우미 메서드가 있습니다. update_syndicate 메서드 내에서 이것을 사용하여 게시물이 Gutenberg를 사용하는지 여부를 결정할 수 있다고 생각합니다.
• 그렇다면 rest_after_insert_{$post_type} 연결된 동일한 update_syndicate 메서드를 호출하는 새 작업을 추가할 수 있습니다. 그런 다음 일찍 돌아와서 나머지 코드가 실행되지 않습니다(그리고 이중 업데이트를 받지 않습니다).
• 또한 실행 중인 작업을 확인해야 하므로 rest_after_insert_{$post_type} 후크에 의해 update_syndicate 메서드가 호출되면 해당 지점에서 모든 코드를 실제로 실행한다는 것을 알게 됩니다. \Distributor\Utils\is_using_gutenberg( $post ) && doing_action( 'save_post' ) 와 같은
• 그리고 이 후크는 서로 다른 인수를 사용하기 때문에(하나는 게시물 ID, 하나는 게시물 객체) 이를 처리해야 합니다. get_post 는 ID나 객체를 허용하므로 get_post( $post_id ) 를 호출하는 것처럼 간단해야 한다고 생각합니다.

이 접근 방식(또는 우려 사항)에 대해 질문이 있으면 알려주십시오. 아직 테스트하지 않았지만 이론상으로는 이것이 작동해야 하고 이중 업데이트가 발생하는 것을 방지해야 한다고 생각합니다.

@dkotter 자세한 답변 감사합니다! 귀하의 제안을 기반으로 수정 사항을 구현하는 데 찔렀고 이전에는 생각하지 못한 또 다른 문제를 발견했습니다. :)

같은 것을 추가

if ( \Distributor\Utils\is_using_gutenberg( $post ) && doing_action( 'save_post' ) ) {
    add_action( "rest_after_insert_{$post->post_type}", array( '\Distributor\InternalConnections\NetworkSiteConnection', 'update_syndicated' ) );
    return;
}

신디케이트된 기사의 업데이트가 레거시 메타박스를 사용하지 않고 클래식 편집기와 Gutenberg 모두에서 작동합니다. 그러나 rest_after_insert_{$post->post_type} 후크를 트리거하지 않는 나머지 요청 실행 후 별도의 사후 요청 실행에 저장되기 때문에 레거시 메타박스와 함께 구텐베르크를 사용할 때 중단됩니다. 즉, 레거시 메타박스의 메타데이터가 업데이트 시 신디케이트되지 않음을 의미합니다.

당신이 말했듯이 이 상황을 처리하는 좋은 방법은 없는 것 같지만 위의 if 문에 ! isset( $_GET['meta-box-loader'] ) 를 추가하면 최소한 작동합니다. 이것은 Gutenberg 및 레거시 메타박스를 사용하는 경우 업데이트된 게시물이 두 번 신디케이트된다는 것을 의미하지만, 부분적으로 신디케이트되는 것보다 낫고 현재 작동하는 방식이기도 합니다.

따라서 이것을 update_syndicated 방법에 추가하면 (테스트할 수 있는 한) 이 작업을 클래식 편집기와 Gutenberg 모두에서 훌륭하게 만들고 레거시 메타박스를 사용하는 Gutenberg에서 "괜찮습니다":

if ( \Distributor\Utils\is_using_gutenberg( $post ) && doing_action( 'save_post' ) && ! isset( $_GET['meta-box-loader'] ) ) {
    add_action( "rest_after_insert_{$post->post_type}", array( '\Distributor\InternalConnections\NetworkSiteConnection', 'update_syndicated' ) );
    return;
}

@dkotter 이 솔루션이 합리적이라고 생각하거나 이 문제를 처리할 다른 방법을 생각할 수 있으면 알려주십시오. 그러면 이에 대한 PR을 할 수 있습니다.

@lakrisgubben 테스트해 주셔서 감사합니다. 나는 Gutenberg가 사용자 정의 메타 상자(다른 프로젝트에서 문제를 일으켰음)가 있는 경우 두 가지 요청을 한다는 것을 알고 있었지만 rest 후크가 이러한 경우에 실행되지 않는다는 것을 깨닫지 못한 것 같습니다. register_post_meta 사용하여 메타를 등록하고 show_in_rest 값이 true인 경우 실행).

meta-box-loader 변수는 사용자 정의 메타를 처리하는 경우 항상 설정되어야 하는 것처럼 보이지만 그것에 의존하는 것은 이상적이지 않습니다. 그러나 이 문제를 해결할 핵심적인 방법은 없는 것 같고 내가 언급한 다른 모든 접근 방식은 훨씬 더 비이상적으로 보이기 때문에 이것이 적절한 방법이라고 생각합니다.

함께 할 시간이 있다면 PR을 테스트하게 되어 기쁩니다. 모든 노력에 감사드립니다!

@dkotter PR 첫 통과: https://github.com/10up/distributor/pull/518