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

Change Breadcrumb name on gallery plugin #2816

Closed
kreossino opened this issue Oct 12, 2017 · 8 comments
Closed

Change Breadcrumb name on gallery plugin #2816

kreossino opened this issue Oct 12, 2017 · 8 comments
Labels
type: bug A problem that should not be happening
Milestone

Comments

@kreossino
Copy link
Member

i have modified in e107_plugins/gallery/template/gallery_template.php
$GALLERY_TEMPLATE['list']['caption'] = LAN_PLUGIN_GALLERY_TITLE;
in
$GALLERY_TEMPLATE['list']['caption'] = LAN_PLUGIN_GALLERY_NEWTITLE;
now in
../e107_plugins/gallery/languages/English_global.php
i have added this row

define("LAN_PLUGIN_GALLERY_NEWTITLE", "Collection");

but the breadcrumb and the catpion don't change i see "gallery" not "Collection"

@tgtje
Copy link
Contributor

tgtje commented Oct 13, 2017

It is not that easy as LAN_PLUGIN_GALLERY_TITLE is used on multiple positions throughout files.. Your case would be using (change aka hacked/renamed plugin) the lang file; template; xml file; admin_gallery; controllers/index and e_shortcode, and some more to reflect each change ( about 15 i guess) The above mentionded is NOT enough.

@kreossino
Copy link
Member Author

ok i do some test, but ok, is a very good solution if in admin-panel of e107 the admin can create a new "media owner"

@Jimmi08
Copy link
Contributor

Jimmi08 commented Oct 13, 2017

media_owner should be plugin name, so you need to set this in plugin.xml

@kreossino
Copy link
Member Author

OK Jimmi08. but I do not know how to do. can you send me an example?

@Jimmi08
Copy link
Contributor

Jimmi08 commented Oct 14, 2017

Hm, I am maybe wrong. Its limited.

		$this->restricted = array(
			"_common" 	=> "_common",
			"_icon"		=> "_icon",
			"news"		=> "news",	
			"page"		=> "page",
			"download"	=> "download"					
		);

@CaMer0n
Copy link
Member

CaMer0n commented Oct 14, 2017

It's not limited, those are the core ones during upgrade. See the banner plugin.xml for an example.

@CaMer0n CaMer0n added the type: bug A problem that should not be happening label Oct 17, 2017
@CaMer0n
Copy link
Member

CaMer0n commented Oct 17, 2017

That should fix it @kreossino . Please let me know how it goes.

@CaMer0n CaMer0n added this to the e107 v2.1.7 milestone Oct 17, 2017
@kreossino
Copy link
Member Author

for me is perfect!!! thank you 😜

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A problem that should not be happening
Projects
None yet
Development

No branches or pull requests

4 participants