Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

{USER_EXTENDED=location.text_value} is not working in forum plugin #3957

Closed
Jimmi08 opened this issue Sep 20, 2019 · 6 comments
Closed

{USER_EXTENDED=location.text_value} is not working in forum plugin #3957

Jimmi08 opened this issue Sep 20, 2019 · 6 comments
Assignees
Labels
plugin: forum status: testing required Someone needs to confirm this issue's existence and write a test to prevent the fix from regressing.

Comments

@Jimmi08
Copy link
Contributor

Jimmi08 commented Sep 20, 2019

Could somebody confirm this?

If you use extended field location, it should be displayed inside
{USERCOMBO}

this way:
$ue = $tp->parseTemplate("{USER_EXTENDED=location.text_value}", true);

it return empty value. string(0) ""

It could be used directly in template too:
{USER_EXTENDED=location.text_value}

In news EUF is used this way:
e107::getParser()->parseTemplate("{USER_EXTENDED={$field}.{$type}.{$userid}}");

so I think this should be fixed in forum plugin

@Moc Moc self-assigned this Sep 20, 2019
@Moc Moc added plugin: forum status: testing required Someone needs to confirm this issue's existence and write a test to prevent the fix from regressing. labels Sep 20, 2019
@Jimmi08
Copy link
Contributor Author

Jimmi08 commented Sep 20, 2019

This works:
<div> {USER_EXTENDED=location.text} : {USER_EXTENDED=location.value}</div>

Now I have a problem with gender:
Options are not translated (neither in admin)
Gender : M => UE_LAN_MALE
After manually changing values in admin area, result is displayed (key and value):
Gender : M => Man
Should those options be changed manually?

@Moc
Copy link
Member

Moc commented Sep 23, 2019

For me, on a clean install, the location EUF is shown in the usercombo. No modifications.

image

@Moc
Copy link
Member

Moc commented Sep 23, 2019

I think your second (separate) issue with the translation of EUF labels and values, is related to this issue, which is a bug: #3881

@Jimmi08
Copy link
Contributor Author

Jimmi08 commented Sep 23, 2019

For me, on a clean install, the location EUF is shown in the usercombo. No modifications.

it is displayed without label. Like with only "value". But it has "text_value" settings and label "Location:" is missing.

@Moc
Copy link
Member

Moc commented Sep 23, 2019

Oops, missed that, I'm sorry, Doing more testing :)

@Moc Moc closed this as completed in 98b0cc6 Sep 23, 2019
@Moc
Copy link
Member

Moc commented Sep 23, 2019

Fixed, was an issue in the user_extended shortcode, not in forum.

The translation issue, I consider a separate issue. Probably #3881. Please check if that is indeed your issue as well, or open a new issue, so we can keep this one clean :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plugin: forum status: testing required Someone needs to confirm this issue's existence and write a test to prevent the fix from regressing.
Projects
None yet
Development

No branches or pull requests

2 participants