Are you planning on porting this module (love it by the way) to D7?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Mark Theunissen’s picture

Yeah, I think so. I won't be doing it until code freeze, but if someone else wants to maintain the port they are welcome to.

EgbertB’s picture

Great. Will be using it then for some of our sites.

arhak’s picture

I would like to see this module going on, but... D7 already? having 40 opened issues?
I would like to see a clearer issue queue before moving on?
will D6 be abandoned?

klonos’s picture

subscribing... so I know when there'll be something available for testing ;)

AgaPe’s picture

Any plans?

clashar’s picture

subscribe

aschiwi’s picture

At least for the node/add pages we don't need betterselect anymore because we can pick to use checkboxes for the field. What we do still need it for is the option to also save the term's parent term and to show checkboxes in Views exposed filter.

klonos’s picture

Title: Port to drupal 7 » Port Better select module to drupal 7
Issue tags: +D7 porting, +port to d7, +d7 ports
1kenthomas’s picture

Here's my suggestion.

If you want a module ported to D7, and you plan on using it on sites, put up a reward bounty :)

K

mgifford’s picture

There are lots of discussions of demand for developers to upgrade modules but almost no time.

Offering a bounty is one way. Heck, setting up a collection pool to do it is another.

Even just starting the ball rolling by pushing this through Coder is a good start.

This is a nice module and as a community we need to figure out how to make our work sustainable.

ipwa’s picture

I might be porting this with @marvil07 for a project we're working on. I don't want to promise anything because we might take a different approach, I'll comment back with a patch if this happens or to let you guys know we took a different approach.

jonskulski’s picture

Status: Active » Needs review
FileSize
6.22 KB

Here is the basic patch for a drupal 7 port. I branched off the latest 6.x-1.x.

aschiwi’s picture

Thanks for starting with this port jonskulski!
I'm no coder so my feedback is not gonna be the best. I think you should patch against the latest dev since it's a lot more recent than the current release. Your patch can't be git applied but applies fine using the regular patch command.

I have a taxonomy which gets checkboxes in node forms with default Drupal 7 behavior. We don't need Betterselect for that anymore. It would be useful for usage in Views exposed filters but activating the patched module does not give me checkboxes there or options in Views to pick checkboxes per Taxonomy.

drupalina’s picture

subscribing.

gawi’s picture

I was looking for this for a long time. Subscribing for a D7 release and ready for testing!

cajmcmahon’s picture

subscribing

designerbrent’s picture

sub

sandie’s picture

subscribing and ready for testing...

ipwa’s picture

We ended up using a hook form alter for what we needed, but glad to see there is now a parch on the queue.

harriszrashid’s picture

subscribing

mgifford’s picture

Testing the patch above:

$ git clone --branch 6.x-1.x http://git.drupal.org/project/betterselect.git
$ cd betterselect
$ git apply betterselect-drupa7port-488520.patch
betterselect-drupa7port-488520.patch:44: trailing whitespace.
function betterselect_element_info() {
betterselect-drupa7port-488520.patch:56: trailing whitespace.
function betterselect_process_element($element, &$form_state, $complete_form) {
warning: 2 lines add whitespace errors.

But I enabled the module and didn't get any errors. Looks like it's ready for others to test.

@jonskulski someone should fix white space errors. So easy to add them when putting in a patch and definitely not critical.

If a tarball would be helpful let me know. Also, if anyone's interested in being a maintainer..... See the project page.

Is there a good example to see this in core? I'm trying to find a page where it should be showing for testing.

eme’s picture

Subscribing...

patrick.thurmond@gmail.com’s picture

Well I got it enabled, but it is not doing anything beyond being enabled. So yeah, this still needs some work. I will see what I can do.

fourmi4x’s picture

Suscribing !

lee20’s picture

Subscribing as well. Would love to see a 7.x branch for this.

klonos’s picture

Title: Port Better select module to drupal 7 » Port Better Select module to drupal 7
doublejosh’s picture

Pointing out similar module for Drupal 7 (from a jQuery plugin) with even more power!
http://drupal.org/project/chosen

aschiwi’s picture

Status: Needs review » Closed (won't fix)

I still don't see what you guys want this for in Drupal 7? On node forms it's Drupal core that offers checkboxes for term selection. In Views you can use Better Exposed Filters to achieve this.

I think this issue can be closed, please reopen if you don't agree.

klonos’s picture

Component: Miscellaneous » Documentation
Category: feature » task
Status: Closed (won't fix) » Active

...at least document this info on the projects page then. That's the first place people will turn to for a solution when upgrading to D7, so placing this in a prominent place would be really helpful. Might also want to link to the Chosen project as a D7 alternative too as Josh pointed back in #27.

mgifford’s picture

Do we need to start looking for a new maintainer? There was a patch from June of last year.

ipwa’s picture

I'll add the patch to have a d7 branch (in case someone wants to provide additional patches) and add the d7 alternatives to the description until we have an official d7 release.

klonos’s picture

Component: Documentation » Code
FileSize
28.63 KB

I'm switching this back to code and kindly request to reconsider things about a D7 port. Besides using this as a field widget, other modules could take advantage of it in their UI. One such example is Fast Permissions Administration where its "Roles" multiselect could be converted to a Better Select widget:

That would spare mobile users from having to figure out a way to ctrl+click on roles listed in that widget and simply use the checkboxes provided by Better Select instead. I simply cannot imagine a way to achieve this with Views + Better Exposed Filters or Chosen.

klonos’s picture

ipwa’s picture

ok I think I'll cave in, next code sprint I attend I will work on this

klonos’s picture

Thanx for reconsidering Nicolas!

I'm not that active in d.o lately and I won't be for some time, but I'll be checking things as often as I make time. Let me know if you've got anything ready for testing. Thanx again ;)

superdorx’s picture

Any news?

Nikit’s picture

Priority: Normal » Major
Status: Active » Needs review
FileSize
7.98 KB

I create patch (merged with jonskulski patch) for porting to Drupal 7.

Sponsored by superdorx.com

John Franklin’s picture

Status: Needs review » Fixed

Rerolled and committed. Thanks for the patches.

klonos’s picture

Version: 6.x-1.x-dev » 7.x-1.0-beta1

Thanx!

Status: Fixed » Closed (fixed)
Issue tags: -D7 porting, -port to d7, -d7 ports

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