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

Adicionando Metabox para tela de edição de terms #318

Merged
merged 6 commits into from
Jul 21, 2015

Conversation

matheusgimenez
Copy link
Member

Conforme issue #311 criei esse pull request

Para salvar os fields, usei a tabela options e suas funções. Pensei ser melhor que criar um nova tabela.

if ( $attachments ) {
foreach ( $attachments as $attachment_id ) {
$html .= sprintf( '<li class="image" data-attachment_id="%1$s">%2$s<ul class="actions"><li><a href="#" class="delete" title="%3$s"><span class="dashicons dashicons-no"></span></a></li></ul></li>',
$attachment_id,https://bay181.mail.live.com/?fid=flinbox
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deixou escapar aqui o seu o endereço do webmail :p

@matheusgimenez
Copy link
Member Author

Sorry pelos erros, fiz essa classe no centro cultural e é dificil concentrar com barulho hehe.. mas pelo menos foi pouca coisa hehe.

Quanto ao prepare, pensei que o $wpdb já fizesse isso por trás, devia ter olhado o codigo antes né hehe.. Vivendo e aprendendo.

Vou corrigir isso tudo.

@claudiosanches
Copy link
Member

@matheusgimenez bom trabalho.
Eu também prefiro fazer isso dentro de options, já que criar outra tabela pode ser bem tenso para temas.
Acabei de dar uns toques ai de como melhorar :)

@claudiosanches
Copy link
Member

@adammacias
Copy link
Member

Menininhos espertos 😄

@adammacias adammacias added this to the v2.8.0 milestone Jul 20, 2015
@matheusgimenez
Copy link
Member Author

Creio que resolvi todos

Abraços

@claudiosanches
Copy link
Member

Se bem que nesse caso fica melhor com:

        $wpdb->get_results( $wpdb->prepare( "
            DELETE
            FROM $wpdb->options
            WHERE option_name
            LIKE %%%s%%
         ", $wpdb->esc_like( $option ) ),
        );

@matheusgimenez
Copy link
Member Author

@claudiosmweb

Tentei isso:

$wpdb->get_results(
    $wpdb->prepare(
        "
        DELETE
        FROM $wpdb->options
        WHERE option_name
        LIKE %%%s%%
        ",
        $wpdb->esc_like( $option )
    )
);

Mas não deletou :(

Dei um echo na string e o resultado é o seguinte:

DELETE
FROM wp_10_options
WHERE option_name
LIKE %odin\\_term\\_meta\\_10%

Testando no PHPMyAdmin temos o seguinte erro:

Parece haver um erro na sua query SQL. A saída de erro do servidor MySQL abaixo, se houver alguma, também poderá ajudar a diagnosticar o problema.

ERROR: String de pontuação desconhecida @ 55
STR: \\
SQL: DELETE FROM wp_10_options WHERE option_name LIKE %odin\\_term\\_meta\\_10%


Query SQL:

DELETE FROM wp_10_options WHERE option_name LIKE %odin\\_term\\_meta\\_10%

Mensagem do MySQL: Documentação
#1064 - You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '%odin\\_term\\_meta\\_10%' at line 1 

@matheusgimenez
Copy link
Member Author

Consegui aplicar o escape agora! Acho que está tudo ok.

Abraços

@claudiosanches
Copy link
Member

Aee o/

claudiosanches added a commit that referenced this pull request Jul 21, 2015
Adicionando Metabox para tela de edição de terms
@claudiosanches claudiosanches merged commit 55247cb into wpbrasil:master Jul 21, 2015
@claudiosanches
Copy link
Member

@matheusgimenez valeu cara :)

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.

3 participants