Here's the issue I'm getting with a fresh HEAD and field_group 8.x-1.x-dev when trying to enable the module
Recoverable fatal error: Argument 3 passed to Drupal\Core\Plugin\DefaultPluginManager::__construct() must implement interface Drupal\Core\Extension\ModuleHandlerInterface, none given, called in /var/www/html/head/modules/field_group/lib/Drupal/field_group/Plugin/FieldGroupPluginManager.php on line 30 and defined in Drupal\Core\Plugin\DefaultPluginManager->__construct() (line 115 of core/lib/Drupal/Core/Plugin/DefaultPluginManager.php).
Seems like the latest dev snapshot is very outdated (Oct. 2013) so it's likely that's the module is using deprecated code. Didn't yet spend time searching what code exactly.
Comment | File | Size | Author |
---|---|---|---|
#7 | field-group-third-party-settings-2258491.patch | 7.48 KB | webflo |
Comments
Comment #1
nils.destoop CreditAttribution: nils.destoop commentedthe 8.x branch is more testing code by Hydra. It was never a working version. Changing the title of the issue.
Comment #2
Anonymous (not verified) CreditAttribution: Anonymous commentedI'm looking forward for d8 verison of this module.
Comment #3
miro_dietikerI see zuuperman was active 6 weeks ago and pushed things forward.
Would be awesome to have some status update here. :-)
Comment #4
webflo CreditAttribution: webflo commentedSprinted two days together with Hydra in Berlin. The code in on github https://github.com/webflo/field_group
Lets talk in Amsterdam :)
Comment #5
nils.destoop CreditAttribution: nils.destoop commentedOk, i'll see you there :). Just pushed latest version. It completely moves formatters, formatter settings, formatter summary to the plugin system.
Comment #6
miro_dietikerWow, awesome guys! Thank you for pushing this forward!
Comment #7
webflo CreditAttribution: webflo commentedWe decided during DrupalCon to use
ThirdPartySettings
onEntity(View/Form)Display
as storage. The following patch is a first step in this direction.Comment #8
Stalski CreditAttribution: Stalski commentedTrue, however, if we would need a displayComponentHandler for some other reason, then the storage would be there.
So ok, we certainly start with this and it might stay this way.
Comment #9
nils.destoop CreditAttribution: nils.destoop commentedLooks good, committed it to current dev. If #1875974: Abstract 'component type' specific code out of EntityDisplay will be committed, this probably will move to there.
Comment #10
miro_dietikerzuuperman, Stalski, please update the issue here once persistency is stable enough to start using field_group. Is there a timeline / roadmap for 8.x?
Comment #11
miro_dietikerWhat is the current status of the field_group storage stability? (Any current work going on?)
Also no activity for 24 days in #1875974: Abstract 'component type' specific code out of EntityDisplay
Comment #12
MrHaroldA CreditAttribution: MrHaroldA commentedI've wondered why field_group wasn't in D8 core ... this is a must-have for every site out there that has a front-end, or backend.
So, what can we do to help? The current state isn't even test-worthy, as it crashes everywhere and doesn't show up anywhere.
Comment #13
miro_dietikerMrHaroldA: There will always be top X modules that should have been in core.
Obviously it wasn't important enough. Otherwise someone would have taken the ball...
In the current status of Drupal core, module maintainers still need to frequently update contrib code to make it work again.
If you want to help, join a sprint, provide pull requests (many change notices from core would be easy to apply) or fund developers that do the hard work for you.
Comment #14
MrHaroldA CreditAttribution: MrHaroldA commentedWhere's the latest code? Github isn't touched in over 2 months and the d.o repository has been idle for 7 weeks.
Comment #15
webflo CreditAttribution: webflo commentedThe latest code is in https://github.com/webflo/field_group/commits/8.x-1.x-drupal.org (8 days old)
Comment #16
nils.destoop CreditAttribution: nils.destoop commentedThe latest commits of webflo are merged in current dev. I also fixed some extra fatal errors that occurred due core changes. Seems functional again with current dev version of D8.
About the storage: There is idd not much activity in #1875974: Abstract 'component type' specific code out of EntityDisplay, and i don't take it will be finished at D8 release. So the storage will probably remain the same like on current dev version (inside ThirdPartySettings of the display object).
Comment #17
nils.destoop CreditAttribution: nils.destoop commentedExtra update: Div wil probably be removed. With HTML element, you can also add divs.
Comment #18
MrHaroldA CreditAttribution: MrHaroldA commentedI just tried the new dev version ... I'll try to debug this one first:
Comment #19
BerdirSo, we're starting to actually use this. testing it, random feedback and thoughts:
1. The edit icon is broken, I assume because of https://www.drupal.org/node/2384159 ?
2. I think we should consider switching adding groups the same way that fields now do as well, the common local action pattern. It is a click more, but it gives more options there, like being able to immediately load the plugin settings through ajax, instead of having to search for them again. I can also imagine multiple options to solve the "add something at a specific place" use case, like a parent selection on the add, and/or a "Add tab/item" link next to delete that preselects the correct type and parent already.
3. Not sure if it is supposed to work, but machine name generation does not (would be easy to fix with 2.)
4. The types should be sorted alphabetically and could really use some description/help (again 2. could help with that)
5. the default advanced group/thing is still/even more a blackbox, any idea how to improve that? At least as annoying as in 7.x that you can't put anything into the sidebar..
6. The add group form is inside/below "Disabled", another thing that would be solved by 2. :)
7. D8 supports #group to move form elements into fieldsets/details, see how NodeForm does that for example with uid/created. No idea if that works for view as well, but maybe that would be a way to simplify the whole prerender thing?
We might have someone that can work on some of those things, but I wanted to get some feedback first.
Comment #20
miro_dietikerWe completed our review and compared a field_group based approach versus an alter code based group approach.
We realised that the technical dept (volatility, limitations) and extra work (to make it appear similar as core) is too high with the current state.
One of the key reasons for our decision is also that site builder editability is not soo important in our current project. Also migrating away from the small code fragments is easy.
Thus, we decided for the coded approach (for now).
For other projects (soon) this might be different. So we are still eager to push this to a decent point where we can make it a default D8 recommendation.
Looking forward to further feedback.
Comment #21
askibinski CreditAttribution: askibinski commentedIt might be the field_group module becomes obsolete when this issue gets in Drupal 8.1.x #607396: Killer feature: Fieldable Fields in core
Comment #22
puddyglumI'm willing to contribute to getting this ready for d8 when it's released. I just need somebody to tell me it's not a waste of time ;)
Comment #23
miro_dietiker@askibinski: Relying on this issue and all the related things opens a can of worms. We try to make the loose ends work, but i'm not so sure this will happen soon, at least not in 8.0.x core.
So yes, for Drupal 8.0.x we will need a port for field_groups otherwise no grouping / hierarchy of fields is possible in the UI is possible through UI.
Comment #24
puddyglumDo you mind clarifying this?
Comment #25
puddyglumThere's a working version of this at https://github.com/webflo/field_group.git, which I've forked at https://github.com/jmonkfish/field_group.git (make sure to use the 8.x-1.x-drupal.org branch)
Comment #26
DuaelFrWhat's the state of this port?
Is it usable in beta11?
Comment #27
miro_dietikerBoth branches are ~9 months outdated. 99% sure they don't work with beta11 or current HEAD.
Comment #28
DuaelFr@jmonkfish 8.x-1.x-drupal.org branch is only 2/3 months old, that's why I ask ;)
Comment #29
webflo CreditAttribution: webflo at UEBERBIT GmbH commented@jmonkfish Thanks. I merged you changes and remove the 8.x-1.x-drupal.org branch in my repo the reduce the confusion. https://github.com/webflo/field_group is update to date and should work with beta11.
Comment #30
Anonymous (not verified) CreditAttribution: Anonymous at Chapter Three commented@webflo I've tested the latest in your repo against beta12, and it seems that the field group pane doesn't display on the 'manage display' tab, only the 'manage form display'.
I wasn't able to find any valuable information other than the behavior. Let me know if I can help gather any more info.
Comment #31
webflo CreditAttribution: webflo at UEBERBIT GmbH commented@drewbolles Thanks for the bug report. I pushed some important fixed to my github repo. Please try again ..
Comment #32
vorapoap CreditAttribution: vorapoap commented@webflo Thank your for the code
May somone here merge the latest code and release it here in drupal.org.. So it will be used by greater numbers of audience. The recent. Drupal8 beta version is pretty solid now.
Comment #33
Anonymous (not verified) CreditAttribution: Anonymous at Chapter Three commentedI can now see and use the field group on the manage display tab, but the field group doesn't output. I just see my fields without the grouping displayed around them. I don't see any PHP errors regarding field group.
Comment #34
nils.destoop CreditAttribution: nils.destoop commentedA quick update: i just merged webflo-fg/8.x-1.x with current drupal.org 8.x-1.x. Now all the latest code of both repos are in the same branch on drupal.org. There were some commits on drupal.org that didn't exist yet on the webflo repo.
Comment #35
miro_dietikerAwesome! I think we can close this issue then and switch to an issue based workflow.