Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Are you planning on porting this module (love it by the way) to D7?
Comment | File | Size | Author |
---|---|---|---|
#37 | betterselect-488520-12.patch | 7.98 KB | Nikit |
#32 | fpa_needs_better_select.png | 28.63 KB | klonos |
#12 | betterselect-drupa7port-488520.patch | 6.22 KB | jonskulski |
Comments
Comment #1
Mark Theunissen CreditAttribution: Mark Theunissen commentedYeah, 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.
Comment #2
EgbertB CreditAttribution: EgbertB commentedGreat. Will be using it then for some of our sites.
Comment #3
arhak CreditAttribution: arhak commentedI 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?
Comment #4
klonossubscribing... so I know when there'll be something available for testing ;)
Comment #5
AgaPe CreditAttribution: AgaPe commentedAny plans?
Comment #6
clashar CreditAttribution: clashar commentedsubscribe
Comment #7
aschiwi CreditAttribution: aschiwi commentedAt 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.
Comment #8
klonosComment #9
1kenthomas CreditAttribution: 1kenthomas commentedHere'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
Comment #10
mgiffordThere 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.
Comment #11
ipwa CreditAttribution: ipwa commentedI 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.
Comment #12
jonskulski CreditAttribution: jonskulski commentedHere is the basic patch for a drupal 7 port. I branched off the latest 6.x-1.x.
Comment #13
aschiwi CreditAttribution: aschiwi commentedThanks 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.
Comment #14
drupalina CreditAttribution: drupalina commentedsubscribing.
Comment #15
gawi CreditAttribution: gawi commentedI was looking for this for a long time. Subscribing for a D7 release and ready for testing!
Comment #16
cajmcmahon CreditAttribution: cajmcmahon commentedsubscribing
Comment #17
designerbrent CreditAttribution: designerbrent commentedsub
Comment #18
sandie CreditAttribution: sandie commentedsubscribing and ready for testing...
Comment #19
ipwa CreditAttribution: ipwa commentedWe ended up using a hook form alter for what we needed, but glad to see there is now a parch on the queue.
Comment #20
harriszrashid CreditAttribution: harriszrashid commentedsubscribing
Comment #21
mgiffordTesting 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.
Comment #22
eme CreditAttribution: eme commentedSubscribing...
Comment #23
patrick.thurmond@gmail.comWell 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.
Comment #24
fourmi4x CreditAttribution: fourmi4x commentedSuscribing !
Comment #25
lee20 CreditAttribution: lee20 commentedSubscribing as well. Would love to see a 7.x branch for this.
Comment #26
klonosComment #27
doublejosh CreditAttribution: doublejosh commentedPointing out similar module for Drupal 7 (from a jQuery plugin) with even more power!
http://drupal.org/project/chosen
Comment #28
aschiwi CreditAttribution: aschiwi commentedI 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.
Comment #29
klonos...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.
Comment #30
mgiffordDo we need to start looking for a new maintainer? There was a patch from June of last year.
Comment #31
ipwa CreditAttribution: ipwa commentedI'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.
Comment #32
klonosI'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.
Comment #33
klonos...btw:
#1766056: Consider converting the "Roles" multiselect to a betterselect...
Comment #34
ipwa CreditAttribution: ipwa commentedok I think I'll cave in, next code sprint I attend I will work on this
Comment #35
klonosThanx 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 ;)
Comment #36
superdorx CreditAttribution: superdorx commentedAny news?
Comment #37
Nikit CreditAttribution: Nikit commentedI create patch (merged with jonskulski patch) for porting to Drupal 7.
Sponsored by superdorx.com
Comment #38
John Franklin CreditAttribution: John Franklin commentedRerolled and committed. Thanks for the patches.
Comment #39
klonosThanx!