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.Steps to reproduce on a fresh install of Drupal Commons:
Go to Events -> List an Event (node/add/event)
Fill in Title and Description
For Location, select "Online" (the address fields will disappear as expected)
Press Save.
You'll receive these errors:
Address 1 field is required.
City field is required.
State field is required.
ZIP code field is required.
As #2412849: Addressfield is required on user registration form even if checked as non required explains, an optional address must be set to None in order to pass validation, and the default country of United States, hidden as it is and not sent to none, fails.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | drupal-commons-2640042-unrequire-address-for-online-4.patch | 2.08 KB | mlncn |
| #4 | drupal-commons-2640042-unrequire-address-for-online-3.patch | 1.01 KB | mlncn |
| #3 | drupal-commons-2640042-unrequire-address-for-online-2.patch | 671 bytes | mlncn |
Comments
Comment #2
mlncn CreditAttribution: mlncn at Agaric commentedExcited to see that Social for Drupal 8 looks to pick up where Commons left off. In the meantime, patch will be incoming for commons_events.
Comment #3
mlncn CreditAttribution: mlncn at Agaric commentedThis does it: When the location is changed to Online, set the Country of the addressfield (simultaneously hidden by States API) to none (''). People can now save online events. Christmas is saved.
Comment #4
mlncn CreditAttribution: mlncn at Agaric commentedSlight improvement; set the country back to the US if people change back from Online.
Comment #5
mlncn CreditAttribution: mlncn at Agaric for National Institute for Children's Health Quality commentedHere's the final offer: set the country back to whatever the default country is when changing back from Online to Physical or Physical and Online.
Note: SimplyTest.me doesn't work because it provides OpCache memory of 64 which "is not enough space to successfully install Drupal Commons. Change the capacity to at least 96."
But presumably this means it is able to apply the patch to Commons, and i've tested thoroughly. Please review!
Comment #6
eellingwood@nichq.org CreditAttribution: eellingwood@nichq.org commentedWorked for us! We're using this on several production sites.