Closed (fixed)
Project:
Localization server
Version:
7.x-1.x-dev
Component:
l10n_groups
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
30 Oct 2012 at 19:43 UTC
Updated:
8 Mar 2013 at 21:00 UTC
Jump to comment: Most recent file
Comments
Comment #1
SebCorbin commentedCan you post the versions of you modules? L10n_server and OG are the most important ones.
Comment #2
inkling commentedcore: 7.16 (but had the same problem with 7.15)
l10n_server: 7.x-1.x-dev
og: 7.x-2.0-beta2
Entity API: 7.x-1.0-rc3
chaos tool suite: 7.x-1.2+6-dev
Plural formula configurator: 7.x-1.2
Translation template extractor 7.x-1.0
...and various other modules that I don't expect to be related.
Looking at og, I could try 7.x-2.x-dev
Comment #3
inkling commentedI just downloaded and installed og 7.x-2.x-dev, but that made no difference.
Comment #4
pablov2 commentedCore: 7.17
Organic groups Organic groups (og) Module Enabled 7.x-2.0-beta2
Localization server Localization server (l10n_server) Module Enabled 7.x-1.x-dev
Same problem
Comment #5
steinmb commentedThe problem is this code in hook_form().
First line fails,
<?php $body = $node->body[LANGUAGE_NONE][0];due to that the $node object does not contain the body and rying to save save the node description fails since the $body variable is empty. I'll have not looked into the code creating the content type (node bundle) but I'll guess it simply fail in creating the content type properly. Still it is a bit strange since the body field is created on default.
Escalate this issue to critical then I consider, not being able to join just that.
Comment #6
steinmb commented#1826478: Undefined property: stdClass::$body sounds like part of the same issue.
Comment #7
SebCorbin commentedCode from the D6 version is pretty ugly considering how fields are handled in D7. We need to move the l10n logic (i.e. $node->l10n_group_language property) to form alter instead of hook_form in l10n_groups.
Comment #8
SebCorbin commentedHere's a patch:
This patch is part of the #1day1patch initiative.
Comment #9
SebCorbin commentedI've given up on implementing hook_update_N(), anyway we don't have any release yet...
drupal_set_message(t('Remember to configure plurals for the language you mean to create a group for.'), 'warning');because this is why you get a 404.
Will commit if testbot is ok with that
Comment #10
SebCorbin commentedComment #11
gábor hojtsyWoot :) SebCorbin++ :)
Comment #12
inkling commentedWhat Gábor said: "Woot." :-)
I put this on hold long ago and it will take me some time to review this.
Comment #13
inkling commentedWell...I set up Spanish as before. Only now I'm not seeing a Join button. I'm still relatively new to Drupal so I I'm guessing I'm missing something.
Comment #14
SebCorbin commentedI just added OG UI as a dependency of l10n_groups to make sure subscribe links work
Permissions may be the thing you forgot, this should be documented
Comment #15
inkling commentedYou're right. I was missing OG UI from l10n_groups. I was also missing permissions. The Join button appears, once I go to the Spanish group. :-)
However, pressing Join gives me a blank page at ?q=group/node/1/subscribe&destination=translate/languages/es. If this is a separate problem, I'll open a new issue. And if the solution is not immediately obvious to you, I'll try to dig and see what is causing the problem.
I am having problems trying to make an l10n server instance per the instructions in #14, in order to reduplicate the error there. The Quick navigation box says, "No languages with plural formulas defined or no projects scanned yet. Check the configuration of languages, list of connectors and projects." If I check projects, I get a message that no projects are defined yet. That doesn't come as a surprise because I didn't set it up. If I check strings, I get a message that there are no strings to translate. That's no surprise, because I didn't see any strings get downloaded when I picked Spanish. I think I would end up spending a lot of time setting things up in a sandbox that quickly goes away. The l10n server I set up on my localhost is more solid.
Comment #16
inkling commentedI'm still getting the problem with the White Screen of Death. Since this is fixed with the patch above, I opened a new issue, #1925032 Localization Server, Join Group sends me to the White Screen of Death.
Comment #17.0
(not verified) commentedChanged opinion of possible related bug.