e107inc / e107

e107 Bootstrap CMS (Content Management System) v2 with PHP, MySQL, HTML5, jQuery and Twitter Bootstrap. Issue Discussion Room: https://gitter.im/e107inc/e107
https://e107.org
GNU General Public License v3.0
321 stars 214 forks source link

tinymce plugin #2985

Closed Jimmi08 closed 5 years ago

Jimmi08 commented 6 years ago

Latest github, result after installing plugin. Only English site (one language)

image

Evgura commented 6 years ago

e107 Версия 2.1.8 (git) Everything is normal. Russian language is. 34560

Jimmi08 commented 6 years ago

Do you have Russian and English languages or only Russian?

Evgura commented 6 years ago

Is and English. 345600

Jimmi08 commented 6 years ago

It works for me on multilang site too. But not on only English site.

And Php 7.0

Evgura commented 6 years ago

PHP 5.4.45

CaMer0n commented 6 years ago

@Jimmi08 I am unable to reproduce it either. Did you try Debugging 'Includes' ?

Evgura commented 6 years ago

@Jimmi08 You have a refresh icon at the top, try to update the database.

Jimmi08 commented 6 years ago

@214945 Core db is updated, that icon is from plugin I am working on.

image /e107_plugins/tinymce4/admin_config.php /e107_plugins/tinymce4/e_header.php

English_admin.php it's not listed in included files.

It's not important now (not have time) but I will look for more details.

tgtje commented 6 years ago

The post refers to a locale folder( which github file does not have; nor in translations) Sorry mentioned it before in relationship, yes or no locale or root...

tgtje commented 6 years ago

Just adding some testing.... (2.1.7/2.1.8) Notices : Notice Use of undefined constant TMCEALAN_1 - assumed 'TMCEALAN_1', Line 1403 of XXXXXX \e107_handlers\admin_ui.php (2; 3;4 5 and 6 too).. Further testing showed me this : use of lan call : e107::lan('tinymce4','admin', 'true'); in admin_config tiny

TRUE is surrounded with quote (single prime)...But i am unsure (tests do give good results...afaiks) when set to true without quote OR commented the line out completly> all works as should (either english only, or added language). Maybe (just maybe) the lan call is in this case of no use, but interfers in certain conditions?) (found 3 similar contructions of this lan call 'true' but in 'func construct' (banner and news/ commented in tagcloud) (can not replicate issue)

Edit some extra : (debug info Couldn't load language file: ../../e107_plugins/featurebox/languages/English/English_global.php e107::includeLan()() XXX\v218beta\e107_handlers\e107_class.php 3192 Couldn't load language file: ../../e107_plugins/rss_menu/languages/English/English_global.php e107::includeLan()() XXXs\v218beta\e107_handlers\e107_class.php
same conflict if you like > there is no locale folder' adding them does eliminate the debug/notice) related ? #2870

CaMer0n commented 6 years ago

Corrected the 'true' in quotes. @Jimmi08 does it help?

Jimmi08 commented 6 years ago

the same. Nevermind, it's not so important (looks like only I have problem) But I noticed this: image

If I click repair plugin setting, it says Repaired (green message) but that icon is still there. Reinstallation didn't help.

I will debug this when I will time.