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

[CLASSE METABOX] Problema ao setar checkbox com atributo "checked" por padrão #124

Closed
fdaciuk opened this issue Dec 14, 2013 · 11 comments
Closed
Labels

Comments

@fdaciuk
Copy link
Contributor

fdaciuk commented Dec 14, 2013

Ao setar o atributo default como 1 na criação do checkbox, ele sempre ficava marcado após salvar o post, ainda que desmarcasse, pois os valores da função checked() estavam invertidos.

Ajustado nesse commit.

@fdaciuk fdaciuk closed this as completed Dec 14, 2013
@fdaciuk
Copy link
Contributor Author

fdaciuk commented Dec 14, 2013

Na verdade o valor está correto. O problema é que, se o checkbox não for checkado, ele sempre vai comparar com o valor default e checkar.

Estou ajustando isso.

@fdaciuk fdaciuk reopened this Dec 14, 2013
@claudiosanches
Copy link
Member

Então o padrão volta a ficar com ele.
E ai no caso da check de novo.
Precisamos escolher algo melhor para isso xD

@fdaciuk
Copy link
Contributor Author

fdaciuk commented Dec 14, 2013

Exatamente isso que tá rolando. Tô tentando pensar em algumas possibilidades aqui xD

@claudiosanches
Copy link
Member

Na real o interessante é que esse default não seja usado por quando o campo esta vázio.
E sim que rode uma vez para instalar as opções.
Talvez na ativação do tema ou algo assim.

@fdaciuk
Copy link
Contributor Author

fdaciuk commented Dec 14, 2013

Então.. pensei em testar no $_POST, mas também não vai rolar, pois se o usuário acessar o post ele já vai trazer checkado de qualquer forma, pois não tem $_POST.

Acho que teria que gravar duas informações para esse caso: uma para saber se o checkbox vai ser checkado e outra para saber se o valor já foi salvo alguma vez.

@claudiosanches
Copy link
Member

Ele só serve ai para a primeira vez xD
Não vai ter sentido um checkbox que fica sempre marcado mesmo.
Estou fazendo umas coisas aqui e atualizando plugins para o WC 2.1, depois vejo se consigo pensar em algo também.

@ghost ghost assigned fdaciuk Jan 23, 2014
@fdaciuk
Copy link
Contributor Author

fdaciuk commented May 3, 2014

Acho que descobri uma solução, só vou comentar aqui pra lembrar de fazer depois xD

Daria pra adicionar um hidden em toda metabox criada que vem por padrão com valor zero. Esse hidden vai servir para dizer se o post já foi salvo alguma vez. Assim que salvar o post pela primeira vez, o hidden fica com valor 1 e permanece com esse valor.

Aí dá pra testar o valor do checkbox a partir desse hidden. O que acha @claudiosmweb?

@romuloctba
Copy link
Contributor

+1

@fdaciuk
Copy link
Contributor Author

fdaciuk commented Oct 27, 2015

Coloquei pra eu fazer mas acabei ficando sem tempo e nem mexi nisso.. se alguém quiser tentar :)

@romuloctba
Copy link
Contributor

Pretendo tendar depois, nao vou prometer :P qual a solução q vc acha q achou?

@fdaciuk
Copy link
Contributor Author

fdaciuk commented Oct 27, 2015

A do hidden no metabox: #124 (comment)

Pode ser que funcione xD

@fdaciuk fdaciuk removed their assignment Oct 27, 2015
claudiosanches added a commit that referenced this issue Jan 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants