Download & Extend

Port Better Select module to drupal 7

Project:Better Select
Version:6.x-1.x-dev
Component:Miscellaneous
Category:feature request
Priority:normal
Assigned:Unassigned
Status:needs review
Issue tags:D7 porting, d7 ports, port to d7

Issue Summary

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

Comments

#1

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.

#2

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

#3

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?

#4

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

#5

Any plans?

#6

subscribe

#7

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.

#8

Title:Port to drupal 7» Port Better select module to drupal 7

#9

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

#10

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.

#11

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.

#12

Status:active» needs review

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

AttachmentSize
betterselect-drupa7port-488520.patch 6.22 KB

#13

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.

#14

subscribing.

#15

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

#16

subscribing

#17

sub

#18

subscribing and ready for testing...

#19

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

#20

subscribing

#21

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.

#22

Subscribing...

#23

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.

#24

Suscribing !

#25

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

#26

Title:Port Better select module to drupal 7» Port Better Select module to drupal 7
nobody click here