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

load bbcode buttons only if tinymce is used #4708

Closed
Jimmi08 opened this issue Mar 19, 2022 · 0 comments · Fixed by #4732
Closed

load bbcode buttons only if tinymce is used #4708

Jimmi08 opened this issue Mar 19, 2022 · 0 comments · Fixed by #4732
Assignees
Labels
type: enhancement An improvement or new feature request
Milestone

Comments

@Jimmi08
Copy link
Contributor

Jimmi08 commented Mar 19, 2022

Motivation

Saving time
it is parsing bbcode template and bbcode shortcodes without any use.

Proposed Solution

EDITED: test if the user can use HTML at all and tinymce is fired, only then use bbcodes

file: form_handler.php
method: bbarea()

		if (!check_class(e107::getConfig()->get('post_html', e_UC_MAINADMIN))) 
		{
			$ret .=	e107::getBB()->renderButtons($template,$help_tagid);
		}
@Jimmi08 Jimmi08 added the type: enhancement An improvement or new feature request label Mar 19, 2022
CaMer0n added a commit that referenced this issue Apr 1, 2022
…L-is-allowed-for-user

Fixes #4708 load bbcode buttons only if posting HTML is allowed for user
@CaMer0n CaMer0n added this to the e107 2.3.2 milestone Apr 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement An improvement or new feature request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants