• Resolved jnorell

    (@jnorell)


    Hello,

    It seems almost every wordpress site but the most basic are frustrating when it comes to achieving the desired url structure, and I’m trying to get that to my liking with wpadverts pages now. I know what I’d like to accomplish, which is:

    /classifieds/ <— top level [adverts_list] (all categories)
    /classifieds/categoryX/ <— category archives one level deeper
    /classifieds/categoryX/ad-title-1/ <— individual advertisements

    For sub categories I wouldn’t care if they are:

    /classifieds/categoryX/subcatY/
    or just:
    /classifieds/subcatY/

    Seems simple enough. ?? Would you have either pointers to how to achieve that, or even the specific code to achieve that result? I have some pieces working, but not all of it. I suspect it would be a nice snippet to publish for others as well.

    Thanks!

Viewing 5 replies - 1 through 5 (of 5 total)
  • Thread Starter jnorell

    (@jnorell)

    After some hours I have this close to how I want it, still working out any corner cases, etc. I’ll post details when it’s finished.

    Plugin Author Greg Winiarski

    (@gwin)

    Hi, ok thanks for the update, once you will have the code for this finished i would appreciate if you would paste it here, maybe someone will find it useful.

    One other way to setup custom permalinks is to use a plugin like https://www.ads-software.com/plugins/custom-post-type-permalinks/ but i cannot tell if it will allow to setup permalinks exactly the way you want it.

    Thread Starter jnorell

    (@jnorell)

    Here’s what I have so far, with light testing on basic functionality. I haven’t processed payments or done anything fancy.

    It would be great if you’d consider incorporating this functionality (and other commonly requested snippets) right into the plugin, with a few options to enable/configure it. In this case, change the hard-coded “classifieds” base path to be configurable, and maybe add the ability to specify the ‘add’ and ‘manage’ pages so you don’t have to look that up and hope they haven’t changed names (I have a hard-coded fallback page id, probably not needed).

    In my setup I took the default ‘Adverts’ page which the plugin creates (ie. the page that includes the [adverts_list] shortcode), renamed it to Classifieds, and change the page slug to ‘classifieds’, to that page is visible at /classifieds/ as normal. The Add and Manage pages are then /classifieds/add/ and /classifieds/manage/.

    
    /**
     * Customize advert post type slug
     *
     * @link https://github.com/simpliko/wpadverts-snippets/blob/master/custom-slugs/custom-slugs.php
     */
    add_action( 'adverts_post_type', 'customize_adverts_post_type' );
    function customize_adverts_post_type( $args ) {
    
      if ( ! isset( $args["rewrite"] ) ) {
        $args["rewrite"] = array();
      }
    
      # adverts_post_type hook is used for multiple post types, we only want to affect 'advert'
      if ( ( ! isset( $args['taxonomies'] ) ) || ( ! in_array( 'advert_category', $args['taxonomies'] ) ) ) {
          $args["rewrite"]["with_front"] = false;
          return $args;
      }
    
      $args["rewrite"]["slug"] = "classifieds/%advert_category%";
      $args["rewrite"]["with_front"] = false;
      $args["rewrite"]["feeds"] = false;
      $args["rewrite"]["pages"] = false;
    
      return $args;
    }
    
    /**
     * Customize advert_category taxonomy slug
     *
     * @link https://github.com/simpliko/wpadverts-snippets/blob/master/custom-slugs/custom-slugs.php
     */
    add_action( 'adverts_register_taxonomy', 'customize_adverts_taxonomy' );
    function customize_adverts_taxonomy( $args ) {
      if ( ! isset( $args["rewrite"] ) ) {
        $args["rewrite"] = array();
      }
    
      $args["rewrite"]["slug"] = "classifieds";
      $args["rewrite"]["with_front"] = false;
      $args["rewrite"]["feeds"] = false;
      $args["rewrite"]["pages"] = false;
    
      return $args;
    }
    
    /**
     * Expand %advert_category% in permalink structure.
     *
     * Adapted from https://shibashake.com/wordpress-theme/add-custom-taxonomy-tags-to-your-wordpress-permalinks
     */
    function customize_adverts_category_permalink( $permalink, $post_id, $leavename ) {
        if ( strpos( $permalink, '%advert_category%' ) === FALSE ) return $permalink;
    
        // Get post
        $post = get_post( $post_id );
        if ( ! $post ) return $permalink;
    
        // Get taxonomy terms
        $terms = wp_get_object_terms( $post->ID, 'advert_category' );
        if ( ! is_wp_error( $terms ) && ! empty( $terms ) && is_object( $terms[0] ) ) {
            $taxonomy_slug = $terms[0]->slug;
        } else {
            # we can't just remove the slug or the post name is matched as
            # a category name and we get a not found error
            #return str_replace( '%advert_category%/', '', $permalink );
            
            # try to default to first category in the taxonomy
            $terms = get_terms( array(
                'taxonomy' => 'advert_category',
                'hide_empty' => false,
                'orderby' => 'term_id',
            ) );
            if ( ! empty( $terms ) && ! is_wp_error( $terms ) ) {
                $taxonomy_slug = $terms[0]->slug;
            } else {
                # default category slug
                $taxonomy_slug = 'for-sale';
            }
        }
    
        return str_replace( '%advert_category%', $taxonomy_slug, $permalink );
    }
    add_filter( 'post_type_link', 'customize_adverts_category_permalink', 10, 3 );
    
    /**
     * Set default advert_category for adverts.
     *
     * This function sets a default category for any advert if unset
     * (match the default category set in the permalink above).
     */
    function customize_adverts_default_category( $post_id, $post, $update ) {
        $slug = 'advert';
        if ( $slug != $post->post_type ) {
            return;
        }
    
        $terms = get_terms( array(
            'taxonomy' => 'advert_category',
            'hide_empty' => false,
            'orderby' => 'term_id',
        ) );
        if ( ! empty( $terms ) && ! is_wp_error( $terms ) ) {
            wp_set_object_terms( get_the_ID(), $terms[0]->term_id, 'advert_category' );
        }
    }
    add_action( 'save_post', 'customize_adverts_default_category', 10, 3 );
    
    /**
     * Add rewrite rules for adverts url structure.
     */
    add_action( 'init', 'customize_adverts_rewrite_rules' );
    function customize_adverts_rewrite_rules() {
        global $adverts_rewrite_rules_added;
        $adverts_rewrite_rules_added = 0;
    
    # force flush_rules() for testing (FIXME: disable for production!!!)
    if ( 1 ) {
        global $wp_rewrite;
        $wp_rewrite->flush_rules();
    }
    
        # rewrite rule for per-category feed
    #
    # the generated feed isn't right (the <span> containing each 
    # advert title is outside the <a> link), disabled until fixed.
    #
    #    add_rewrite_rule(
    #        'classifieds/([^/]+)/(feed|rdf|rss|rss2|atom)/?$',
    #        'index.php?advert_category=$matches[1]&feed=$matches[2]',
    #        'top'
    #    );
    #    $adverts_rewrite_rules_added++;
    
        # todo:  add rule for classifieds/feed (all categories)
    
        # rewrite for individual advert
        add_rewrite_rule(
            'classifieds/([^/]+)/(.+)/?$',
            'index.php?advert=$matches[2]',
            'top'
        );
        $adverts_rewrite_rules_added++;
    
        # rewrite for category
        #
        # The 'add' and 'manage' pages would be masked by this rewrite,
        # so we exclude those in the regex: (?!add|manage)
        #
        # get post object for 'Default Ads List Page' set in wpadverts options
        if ( function_exists( 'adverts_config' ) ) {
            $post = get_post( adverts_config( 'config.ads_list_id' ) );
        } else {
            $post = get_page_by_path( 'classifieds' );
    
            if( is_null( $post ) ) {
                $post = get_post( 3917 ); # actual page id via manual lookup
            }
        }
    
        $add_post = null;
        $manage_post = null;
    
        # get post object for 'add' and 'manage' pages
        if( ! is_null( $post ) ) {
            $add_post = get_page_by_path( $post->post_name . '/' . __( 'add' ) );
            if( is_null( $add_post ) ) {
                $add_post = get_post( 3918 ); # actual page id via manual lookup
            }
    
            $manage_post = get_page_by_path( $post->post_name . '/' . __( 'manage' ) );
            if( is_null( $manage_post ) ) {
                $manage_post = get_post( 3919 ); # actual page id via manual lookup
            }
        }
    
        $exclude = 'feed|rdf|rss|rss2|atom';
    
        if( ! is_null( $add_post ) ) {
            $exclude .= '|' . $add_post->post_name;
        } else {
            $exclude .= '|add';
        }
    
        if( ! is_null( $manage_post ) ) {
            $exclude .= '|' . $manage_post->post_name;
        } else {
            $exclude .= '|manage';
        }
    
        add_rewrite_rule(
            'classifieds/((?!' . $exclude . ')[^/]+)/?$',
            'index.php?advert_category=$matches[1]',
            'top'
        );
        $adverts_rewrite_rules_added++;
    
    }
    
    /**
     * Clean up rewrite rules.
     */
    add_filter('rewrite_rules_array', 'customize_adverts_cleanup_rewrite_rules');
    function customize_adverts_cleanup_rewrite_rules( $rules ) {
        global $adverts_rewrite_rules_added;
        $cnt = 0;
    
        foreach ( $rules as $rule => $rewrite ) {
    
             # remove rewrite rules containing '%advert_category%'.
            if ( strpos( $rule, '%advert_category%/') !== false ) {
                unset( $rules[$rule] );
                continue;
            }
    
            # remove all default rules -- no, this kills %adverts-payment% rules
            #if ( strpos( $rule, 'classifieds/') === 0  &&  $adverts_rewrite_rules_added <= $cnt++ ) {
            #    unset( $rules[$rule] );
            #}
    
            # remove conflicting default rule
            if ( $adverts_rewrite_rules_added <= $cnt++ && strpos( $rule, 'classifieds/([^/]+)/?$') === 0 ) {
                    unset( $rules[$rule] );
            }
    
        }
        return $rules;
    }
    
    • This reply was modified 7 years, 7 months ago by jnorell.
    Plugin Author Greg Winiarski

    (@gwin)

    Hi, thanks for the code, i do have some plans to allow users to manage slugs, specifically the “advert” and “advert-category”, but i have this planned for a bigger update, so right now i cannot tell when will this become available in WPAdverts.

    Thread Starter jnorell

    (@jnorell)

    Found a bug in the above customize_adverts_default_category():

    
    function customize_adverts_default_category( $post_id, $post, $update ) {
        $slug = 'advert';
        if ( $slug != $post->post_type ) {
            return;
        }
    
        if ( has_term( '', 'advert_category', get_the_ID() ) ) {
            return;
        }
    
        $terms = get_terms( array(
            'taxonomy' => 'advert_category',
            'hide_empty' => false,
            'orderby' => 'term_id',
        ) );
        
        if ( ! empty( $terms ) && ! is_wp_error( $terms ) ) {
            wp_set_object_terms( get_the_ID(), $terms[0]->term_id, 'advert_category' );
        }
    }
    
Viewing 5 replies - 1 through 5 (of 5 total)
  • The topic ‘url structure’ is closed to new replies.