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

setConfirm and operation's name #137

Closed
uiii opened this issue May 28, 2014 · 4 comments
Closed

setConfirm and operation's name #137

uiii opened this issue May 28, 2014 · 4 comments
Labels

Comments

@uiii
Copy link

uiii commented May 28, 2014

Hi,
I have the following code:

$operations = array('deleteList' => 'Smazat');
$grid->setOperation($operations, function ($operation, $ids) {
    ...
})->setConfirm('deleteList', "...");

But when I perform deleteList action no confirm occures. It is probably the problem with the camel-case naming, because if I change the operation's name in setConfig to delete-list, confirm work fine.

@uiii uiii changed the title setConfirm and operation name setConfirm and operation's name May 28, 2014
@o5 o5 closed this as completed in ead9a40 May 28, 2014
@o5
Copy link
Owner

o5 commented May 28, 2014

Thank you for a report!

@o5 o5 added the BUG label May 28, 2014
@uiii
Copy link
Author

uiii commented May 28, 2014

Sorry, but now the operation's confirmation doesn't work at all. I'm not sure if you did understand the problem. The problem was that for operation with the name deleteList the confirm with name deleteList didn't work but the confirm with name delete-list worked fine. Now, after your commit, any of the confirms doesn't work.

EDIT: I forgot to update grido.js.

@o5
Copy link
Owner

o5 commented May 28, 2014

Are you sure you have an updated grido.js?

@uiii
Copy link
Author

uiii commented May 28, 2014

Ou, sorry, immediately after I posted the message it crossed my mind. Now, after grido.js update, it is all right. Thank for quick replies.

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

2 participants