VBO has the ability to send the checked NIDs to a view page, which is awesome. But it asks the user to enter in the URL of the view page which is not so awesome. Would it be possible for that to be moved into configuration of the field?

Thanks,

Michelle

CommentFileSizeAuthor
#3 pass ids.png73.44 KBbojanz
#3 1306108-pass-ids.patch1.81 KBbojanz

Comments

bojanz’s picture

You can create an advanced action (through the drupal actions ui) where you specify your own label and url.
That action will then turn up separately in the VBO field settings.
That's how people usually use that action. Which in general solves the problem, but I'm not sure how a contrib module
would then include such an advanced action in its own code..

I'd be fine with including the URL in the action settings, and making the action non-configurable.
I'm not sure anyone actually needs the ability to enter the url directly...

michelle’s picture

I'll give your solution a go. In general, though, I can't think of a use case where you would want the choice of page where to send the list of NIDs to be user facing... Seems like most, if not all, of the time you would want that to be a fixed page and just go there rather than having to type in a URL.

Thanks,

Michelle

bojanz’s picture

Status: Active » Needs review
StatusFileSize
new1.81 KB
new73.44 KB

Does this look okay?

michelle’s picture

Did just a quick test and it doesn't work. It tries to go to "http://example.com//348%2C311" and gives this error:

Notice: Undefined index: settings in views_bulk_operations_argument_selector_action() (line 30 of /home/===/example.com/sites/all/modules/views_bulk_operations/actions/argument_selector.action.inc).

I'm heading out the door so can't test more right now to be sure it's not user error.

Michelle

bojanz’s picture

You probably have an old VBO instal, retesting with latest -dev should work.

Is the UX clear enough?

michelle’s picture

Status: Needs review » Reviewed & tested by the community

Yep, that was the problem. That's what I get for trying to test something 3 minutes before I need to be out the door. :)

With the latest dev, it's working perfect. Thanks!

The UI is fine for me but I'm not a noob. A minor enhancement might be to put the part that doesn't need to be put in there at the start to make it clear which bit is the "relative URL" for those who might not know. So like:

http//example.com/ [-----textbox here----]

Michelle

bojanz’s picture

Status: Reviewed & tested by the community » Fixed

Yes, did that (added the url in the prefix, and killed the "relative" word). Committed.

Thanks for testing!

michelle’s picture

Awesome! Thanks so much!

Michelle

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.