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

bbcodes and signature on signup page #3908

Closed
Jimmi08 opened this issue Jul 29, 2019 · 1 comment
Closed

bbcodes and signature on signup page #3908

Jimmi08 opened this issue Jul 29, 2019 · 1 comment
Labels
status: testing required Someone needs to confirm this issue's existence and write a test to prevent the fix from regressing. type: bug A problem that should not be happening
Milestone

Comments

@Jimmi08
Copy link
Contributor

Jimmi08 commented Jul 29, 2019

I can see that there is the template for signature textarea

$BBCODE_TEMPLATE_SIGNATURE = "
	{BB=link}{BB=b}{BB=i}{BB=u}{BB=img}{BB=left}{BB=center}{BB=right}{BB=justify}
	{BB=list}
	<div class='field-spacer'><!-- --></div>
";

$temp['signature'] = $BBCODE_TEMPLATE_SIGNATURE;

But I have custom bbcodes with e_bb.php defined
I don't want to display those bbcodes on the signature field.

It's because using $BBCODE_TEMPLATE for them.
But in new version this is different template then $BBCODE_TEMPLATE_SIGNATURE.
Thanks

@Moc Moc added the status: testing required Someone needs to confirm this issue's existence and write a test to prevent the fix from regressing. label Aug 2, 2019
@Jimmi08 Jimmi08 closed this as completed Mar 19, 2022
@Jimmi08 Jimmi08 reopened this Mar 19, 2022
@Jimmi08
Copy link
Contributor Author

Jimmi08 commented Mar 19, 2022

It wasn't fixed. bbcodes are still used for normal users on the signup page and templating is not working

Fix is:
return e107::getForm()->bbarea('signature', $this->var['user_signature'], 'signature', '', 'small');

@CaMer0n CaMer0n added the type: bug A problem that should not be happening label Mar 27, 2022
@CaMer0n CaMer0n added this to the e107 2.3.2 milestone Mar 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: testing required Someone needs to confirm this issue's existence and write a test to prevent the fix from regressing. type: bug A problem that should not be happening
Projects
None yet
Development

No branches or pull requests

3 participants