WordPress Settings API Wrapper

iamntz · iulie 25, 2014, 5:02pm

Nu mi-a plăcut niciodată API-ul pentru setări din WordPress. Mi se pare greu, peste mână, țâșpe hooks etc. Prin urmare am facut un mic wrapper.

By default adaugă un entry/instanță în meniul Settings (dar s-ar putea modifica ușor să facă un entry nou în sidebar & submenus). De asemenea adaugă hooks ce înregistrează și încarcă scripts & styles doar pe pagina respectivă de opțiuni (util în cazul unul color picker, sortable, tabs etc).

Ceva păreri? Feedback?

github.com

iamntz/wp-settings-wrapper/blob/master/Ntz/Utils/Settings.php

<?php 

namespace Ntz\Utils;

abstract class Settings {
    function __construct( $options = array() )
    {
        $this->options = array_merge( array(
            "settings_title" => "Sample Options",
            "namespace"      => "sample-options",
            "access_level"   => "manage_options"
        ), $options );

        add_action( 'admin_init', array( $this, 'settings_init' ) );
        add_action( 'admin_menu', array( $this, 'admin_menu' ) );

        add_action( 'admin_enqueue_scripts', array( $this, 'register_assets' ) );
        add_action( "admin_print_scripts-settings_page_{$this->options['namespace']}", array( $this, 'load_assets' ) );
    }

This file has been truncated. show original

Exemplu de folosire:

github.com

iamntz/wp-settings-wrapper/blob/master/Sample.php

<?php

class Sample extends \Ntz\Utils\Settings {

    public function register_sections()
    {
        $this->register_section( 'Sample Section 1', "sample-1" );
        $this->register_section( 'Sample Section 2', "sample-2", array( $this, 'section_2_callback') );
    }

    public function section_2_callback( $section ){
      echo "ohai!";
    }


    public function register_fields(){
        $this->add_field(array(
            "section" => 'sample-1',
            "title"   => "Sample Field",
        ));

This file has been truncated. show original

alescx · iulie 25, 2014, 5:30pm

functiile alea ar trebui sa intoarca stringurile, nu sa le printeze.

iamntz · iulie 25, 2014, 5:38pm

Ca să… faci ce cu stringurile?

alescx · iulie 25, 2014, 5:42pm

in 99% din cazuri sa le printezi. dar nu faci output direct din functie.

faci

echo uber_functia_mea().

iamntz · iulie 25, 2014, 6:54pm

De-ar fi așa simplu

add_settings_field, funcția care adaugă field-urile propriu zise, nu întoarce nimic. Și ca să fac toată povestea să întoarcă un string aș complica foarte mult - inutil* aș putea spune - toată povestea.

Cea mai bună chestie pe care aș putea s-o fac ar fi să adaug câteva filtre pe input-uri (deși chiar și asta mi se pare un pic overkill)

Folosesc clasa asta (în diferite versiuni, ce-i drept) de ceva timp. Și pot să-ți spun că am avut nevoie să alterez input-urile fix niciodată.

alescx · iulie 25, 2014, 7:01pm

aaaaaaaaaaaaaa. good guy wordpres. ne invata cum sa nu facem din 2003

iamntz · iulie 25, 2014, 7:21pm

Oh, dar asta e cea mai puțin nasoală chestie!

Ia de aici și te crucește! Nu o dată am avut nevoie doar să extind un pic clasa și de fiecare dată am sfârșit prin a dubla complet metoda start_el doar pentru a face mici ajustări…

IonutBajescu · iulie 25, 2014, 7:37pm

A venit randul meu de data asta

E greu de inteles ce se intampla aici: wp-settings-wrapper/Ntz/Utils/Settings.php at master · iamntz/wp-settings-wrapper · GitHub
Nu ar fii mai simplu cu varianta short, daca tot e un template?

Nu e incomod sa testezi cu output buffer, cand e mai simplu sa testezi direct rezultatul functiei?

github.com

iamntz/wp-settings-wrapper/blob/master/Ntz/Utils/Settings.php#L108


      
              $this->show_help( $help );
          }
          
          
          public function text( $options )
          {
              extract( $options );
              if( !empty( $label ) ) {
                  printf( '<label>%s</label><br>', $label );
              }
              printf( '<input type="text" name="%1$s[%2$s]" value="%3$s" class="regular-text" %4$s />',
                  $this->options['namespace'],
                  $name,
                  isset( $value ) ? $value : esc_attr( $this->get_option( $name ) ),
                  $attrs
              );
          
              $this->show_help( $help );
          }

Unde sunt testele?

Echo vs Return - Functions should never echo code, it should always return. This isn’t as bad as the WordPress 1.x days, but it’s still on my list of issues.
http://milesj.me/blog/read/wordpress-is-bad-mmmk

iamntz · iulie 25, 2014, 7:45pm

Așa, răzbună-te!

Să-ți spun drept, și pentru mine e un pic greu să înțeleg ce se întâmplă acolo. Sunt mizeriile WordPress-ului; asta avem, cu asta defilăm. Nu cred că înțeleg varianta short.

Nu sunt teste pentru că încă încerc să învăț disciplina asta.