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.
I just tried to install your module on d8 and got the following error see attached image for more information.
Fatal error: Class Drupal\coffee\Form\CoffeeConfigurationForm contains 1 abstract method and must therefore be declared abstract or implement the remaining methods
I implemented the extra method in the CoffeeConfigurationForm class see patch
protected function getEditableConfigNames() {
}
Comment | File | Size | Author |
---|---|---|---|
Screenshot from 2015-02-09 19:34:42.png | 99.72 KB | Purencool | |
#2 | coffee-form_error-2422621-2.patch | 885 bytes | yannickoo |
#3 | step-two.png | 28.56 KB | Purencool |
#3 | error-displayed.png | 79.57 KB | Purencool |
#3 | step-one.png | 33.57 KB | Purencool |
Comments
Comment #1
yannickooSorry but your patch includes lots of unnecessary file permission changes. I have added a new patch without these changes. It adds a method comment as well.
Comment #2
yannickooComment #3
Purencool CreditAttribution: Purencool commentedI don't mind fix the patch all you like. I applied coffee-form_error-2422621-2.patch on a new d8 install and it displayed the following error. After I refreshed the page the module says it is installed. See attached images
step-one,
error-displayed,
step-two
Comment #4
yannickooThat issue was already addressed in #2424807: Fix "Can not set values on immutable configuration" :)
Comment #5
Purencool CreditAttribution: Purencool commentedComment #6
michaelmol CreditAttribution: michaelmol commented