The "create a sponsor" form currently includes 2 instances of the Sponsorship Level, one is a required field but the other one is not.

We are currently using the field_sponsor_sponsorship_level field with the Sponsors & Sponsors Admin view.

I would like to propose to remove the extra field_sponsorship_level.

CommentFileSizeAuthor
#6 2012200-cod_sponsors_duplicate_fields-6.patch6.98 KBjyee
#5 2012200-cod_sponsors-rm_sponsorhip_level_field-5.patch4.92 KBDavid Hernández
#2 cod-add-sponsor-without-patch.png25.34 KBrootwork
#2 cod-add-sponsor-with-patch.png20.18 KBrootwork
#1 2012200-cod_sponsors-rm_sponsorhip_level_field-1.patch4.73 KBchrisguindon
screenshot.png161.55 KBchrisguindon

Comments

chrisguindon’s picture

chrisguindon commented
StatusFileSize
new 2012200-cod_sponsors-rm_sponsorhip_level_field-1.patch4.73 KB

Attaching a patch for this.

rootwork’s picture

rootwork
he/him
Primary language English
Location 🇺🇸 US-Pacific 🌎 Chinook (Multnomah, Clackamas) & Cowlitz lands 🌹 Portland, OR
commented
Status: Active » Reviewed & tested by the community
StatusFileSize
new cod-add-sponsor-with-patch.png20.18 KB
new cod-add-sponsor-without-patch.png25.34 KB

Patch applies and works as advertised.

Without patch:
cod-add-sponsor-without-patch.png

With patch:
cod-add-sponsor-with-patch.png

rvilar’s picture

rvilar
Primary language Catalan
Location Montcada i Reixac, Catalonia
commented

The patch in #1 works perfect for me too.

m1n0’s picture

m1n0 commented

#1 works for me too.

David Hernández’s picture

David Hernández commented
Issue summary: View changes
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new 2012200-cod_sponsors-rm_sponsorhip_level_field-5.patch4.92 KB

Quick reroll

jyee’s picture

jyee commented
StatusFileSize
new 2012200-cod_sponsors_duplicate_fields-6.patch6.98 KB

Prior patches remove the "Event(s)" column from the sponsor admin view. This patch preserves that column.

edit: This may not be necessary, as we're moving to relate events directly to sponsors, not through level. (see https://drupal.org/node/2178627)

jyee’s picture

jyee commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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