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.How to reproduce:
Installed Drupal 8 beta10.
Installed coffee via drush en -y coffee (also tried latest version from git)
Cleared caches (drush cr all)
Reloaded the page
Pressed Alt + d
-> Nothing happens. No JS errors
What the problem may be:
$('body').once('coffee', function () { ... } in coffee.js on line 36 is not being executed, even though jquery-once is loaded correctly (and works for other modules)
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | D8.0.gif | 20.3 KB | simone960 |
| #4 | coffee.png | 33.64 KB | axe312 |
| #2 | coffee-fix_once-2486843-2.patch | 790 bytes | yannickoo |
Comments
Comment #1
axe312 CreditAttribution: axe312 at Wunder commentedAny progress here? I am really missing coffee in Drupal 8 :(
Comment #2
yannickooI have found the error in this case, we have to use
.each()from now when using jQuery Once. After applying the patch you should be able to open and use Coffee.Have fun with Coffee for D8 guys :)
Comment #3
yannickooSorry for renaming it again. FYI I have uploaded another patch which fixes the label of node types in the result list: #2559347: Fix node type label.
Comment #4
axe312 CreditAttribution: axe312 at Wunder commentedThx. Works like charm with Drupal 8 - beta 15 and applies on latest coffee dev version.
Comment #5
eloivaquepatch #2 it's works form me on drupal 8 rc1 thanks
Comment #6
simone960 CreditAttribution: simone960 commentedI have applied the patch to D8.0, the search box come out but there's message inside the box
But refreshing the page is the same, not working. Can someone try out and confirm ? Thanks !
Comment #7
yannickooCan you show us the message(s) in your browser's console?
Comment #8
simone960 CreditAttribution: simone960 commentedHere is the one in my console:
This jquery.js is linked to http:// mydomain/core/assets/vendor/jquery/jquery.js
Comment #9
yannickooAre you sure you are using the latest version? I guess this was fixed in #2425397: Fix property name for basePath.
Comment #10
simone960 CreditAttribution: simone960 commentedI'm using the latest version (Date: 2015-Apr-20). The basePath's patch you mentioned was not committed yet to the latest I'm afraid. However, I have applied the patch to the latest and now the message in the console like this:
Drupal 8.0.0 on the remote hosting:
Drupal 8.0.0 on the local computer :
Perhaps it's best that any of you try to install the latest coffee module on the Drupal 8.0.0 and see whether it is really work on the stable release.
Comment #12
willzyx CreditAttribution: willzyx commentedCommitted and pushed to 8.x. Thanks all!
@simone960 try to install the latest DEV version of coffee.
If you think that the problem still exists please open another issue
Comment #13
simone960 CreditAttribution: simone960 commentedThanks ! @willzyx. It works perfectly now !
Comment #15
aiphesSee new issue furthermore.