ACFTimber

Complexity

Total Complexity

10

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	post_get_meta()	0	3	1
A	post_get_meta_field()	0	3	1
A	term_get_meta_field()	0	4	1
A	user_get_meta()	0	3	1
A	user_get_meta_field()	0	3	1
A	__construct()	0	8	1
A	term_set_meta()	0	5	1
A	term_get_meta()	0	15	3

<?php

class ACFTimber {

    function __construct() {

It is generally recommended to explicitly declare the visibility for methods.

        add_filter( 'timber_post_get_meta', array( $this, 'post_get_meta' ), 10, 2 );
        add_filter( 'timber_post_get_meta_field', array( $this, 'post_get_meta_field' ), 10, 3 );
        add_filter( 'timber_term_get_meta', array( $this, 'term_get_meta' ), 10, 3 );
        add_filter( 'timber_term_get_meta_field', array( $this, 'term_get_meta_field' ), 10, 4 );
        add_filter( 'timber_user_get_meta_field_pre', array( $this, 'user_get_meta_field' ), 10, 3 );
        add_filter( 'timber_term_set_meta', array( $this, 'term_set_meta'), 10, 4 );
    }

    function post_get_meta( $customs, $post_id ) {

The parameter $post_id is not used and could be removed.

It is generally recommended to explicitly declare the visibility for methods.

        return $customs;
    }

    function post_get_meta_field( $value, $post_id, $field_name ) {

It is generally recommended to explicitly declare the visibility for methods.

        return get_field( $field_name, $post_id );
    }

    function term_get_meta_field( $value, $term_id, $field_name, $term ) {

It is generally recommended to explicitly declare the visibility for methods.

        $searcher = $term->taxonomy . "_" . $term->ID;
        return get_field( $field_name, $searcher );

$searcher is of type string, but the function expects a boolean.

    }

    function term_set_meta( $value, $field, $term_id, $term ) {

It is generally recommended to explicitly declare the visibility for methods.

        $searcher = $term->taxonomy . "_" . $term->ID;
        update_field( $field, $value, $searcher );

$searcher is of type string, but the function expects a boolean.

        return $value;
    }

    function term_get_meta( $fields, $term_id, $term ) {

It is generally recommended to explicitly declare the visibility for methods.

        $searcher = $term->taxonomy . "_" . $term->ID; // save to a specific category
        $fds = get_fields( $searcher );

$searcher is of type string, but the function expects a boolean.

        if ( is_array( $fds ) ) {
            foreach ( $fds as $key => $value ) {
                $key = preg_replace( '/_/', '', $key, 1 );
                $key = str_replace( $searcher, '', $key );
                $key = preg_replace( '/_/', '', $key, 1 );
                $field = get_field( $key, $searcher );

$searcher is of type string, but the function expects a boolean.

                $fields[$key] = $field;
            }
            $fields = array_merge( $fields, $fds );
        }
        return $fields;
    }

    function user_get_meta( $fields, $user_id ) {

The parameter $user_id is not used and could be removed.

It is generally recommended to explicitly declare the visibility for methods.

        return $fields;
    }

    function user_get_meta_field( $value, $uid, $field ) {

It is generally recommended to explicitly declare the visibility for methods.

        return get_field( $field, 'user_' . $uid );

'user_' . $uid is of type string, but the function expects a boolean.

    }
}