Title says it all.

Consequence: e.g. the node Title field's widget at node/add/article has an "add more" wrapper. Obviously this is unneeded/unwanted.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

The last submitted patch, field_widgets_respect_single_cardinality.patch, failed testing.

Wim Leers’s picture

Status: Needs work » Postponed

Postponed until after dev days to prevent conflicts.

Wim Leers’s picture

Assigned: Wim Leers » Unassigned
swentel’s picture

Status: Postponed » Needs review
FileSize
1.59 KB

We could also just do this. This doesn't at least blow up the submission of a node and we can keep the existing structure. Should normally come back green.

Wim Leers’s picture

Title: Single cardinality fields get multiple cardinality widgets » Only set "add more" wrapper on fields with cardinality >1
Status: Needs review » Reviewed & tested by the community

For the sake of completeness: What I proposed in #1 is unacceptable because internally, everything is indeed handled as if it's multi-valued.

Looks completely sane to me :)

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Nice catch!

Committed and pushed to 8.x. Thanks!

  • Commit 1e04c67 on 8.x by webchick:
    Issue #2226665 by swentel, Wim Leers: Only set 'add more' wrapper on...

Status: Fixed » Closed (fixed)

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

jim_at_miramontes’s picture

Is there a back port of this to D7 (he asks, with a foolish degree of optimism in his voice)? The same problem seems to exist there, but it never really showed up until I started working with Bootstrap 3.x. Just checking; thanks...

b_sharpe’s picture

FileSize
1.68 KB

D7 Version:

dgtlmoon’s picture

Version: 8.0.x-dev » 7.x-dev
Status: Closed (fixed) » Needs review
FileSize
1.55 KB

Re-rolled per current 7.x, seems to still be an issue

  • webchick committed 1e04c67 on 8.3.x
    Issue #2226665 by swentel, Wim Leers: Only set 'add more' wrapper on...

  • webchick committed 1e04c67 on 8.3.x
    Issue #2226665 by swentel, Wim Leers: Only set 'add more' wrapper on...

  • webchick committed 1e04c67 on 8.4.x
    Issue #2226665 by swentel, Wim Leers: Only set 'add more' wrapper on...

  • webchick committed 1e04c67 on 8.4.x
    Issue #2226665 by swentel, Wim Leers: Only set 'add more' wrapper on...