Jump to content
TNG Community
Rob Severijns

Person Map mod error after 13.01 update

Recommended Posts

Rob Severijns

Bart, Ken, Robin or others,

 

This morning I did the update from TNG 13.0 to 13.01.

All previously installed mods work fine after re-installing except for the Person Map mod.

It generates an error on line 83 (see screenshot)

Installing the previous version of the file admin_updatemapconfig.php solves the problem so the error is probably be in the new version of the file although I'm not sure.

image.png

A lot of people use this mod so can anyone solve this error?

Kind regards,

 

Rob

 

Share this post


Link to post
Share on other sites
RickM

I'm sure the Mod authors will be onto it ASAP..

 

Prob not a good idea to revert to a previous version of one file after an upgrade.. You should replace it gain with the 13.0.1 version..

 

Share this post


Link to post
Share on other sites
Rob Severijns

Hi Rick,

I know and I reversed it again.

Was just looking for a possible cause.

Thx anyway for the warning.

Rob

Share this post


Link to post
Share on other sites
Ken Roy

Rob,

I just got up, so you need to give me a chance to install TNG 13.0.1 and analyze the changes.   I will post a new version as soon as I can.

Share this post


Link to post
Share on other sites
Rob Severijns

Morning Ken,

Just enjoy your start of the day.

It is Sunday so no reason to rush things :-)

Rob

Share this post


Link to post
Share on other sites
Ken Roy

Does anyone have the other language files that Bart created for the Person Map mod?  or any language translations that I should include as I update the mod?

 

Share this post


Link to post
Share on other sites
Rob Severijns

Ken,

Will send you the Dutch language file in a sec.

Rob

Share this post


Link to post
Share on other sites
Ken Roy

Rob,

If you send it via my site contact us link, I can also send you the mod to beta test.  So far I have the French language that Bart and I created originally, and Danish from Mogens.

Share this post


Link to post
Share on other sites
Rob Severijns

Ken,

I did send the file as an attachment in a pm.

Will try to send one via your site too.

Rob

Share this post


Link to post
Share on other sites
Ken Roy

Thanks Rob,

I got it just PM a valid email address where I can send you the mod to beta test.

Ken

Share this post


Link to post
Share on other sites
Rob Severijns

Done

Share this post


Link to post
Share on other sites
Ken Roy

Rob,

Thanks for providing the Dutch translations and beta testing.  Person Map v13.1.0.9 has now been posted to the TNG Wiki and includes the French translations that Bart and I initially provided, the Danish translations provided by Mogens Fenger, and the Dutch translations provide by Rob Severijns

Share this post


Link to post
Share on other sites
Ken Roy

Brent Hemphill just sent me the original German and Spanish translation files that Bart had posted.

In TNG 13, I added a collapsible section to the Map Settings for the Person Map mod

Event Map section behavior

therefore, I would like to have the translations for Danish, Dutch, French, German, and Spanish for that text string before I publish an update to add the German and Spanish languages

Thanks

 

Share this post


Link to post
Share on other sites
Ken Roy

Actually, it looks like the translations are there.  I need to figure out why the collapsed header is showing as English text when I switch languages

Share this post


Link to post
Share on other sites
Rob Severijns

Ken,

  1. Is it the text string "Event Map section behavior" you want to have translated? I can't follow the link you provided.
  2. I guess the language switch error isn't in 13.0.1.9 for mine just works fine. (EN <----> NL)

Rob

Share this post


Link to post
Share on other sites
Ken Roy

Rob,

Are you saying that once you switch to Dutch, the Admin > Setup > Map Settings shows the collapsible section as

$admtext['mapheader'] = "Aansturing sectie 'Gebeurteniskaart'";

and not as

$admtext['mapheader'] = "Event Map section behavior";

Share this post


Link to post
Share on other sites
Rob Severijns

Ken,

Yes it does and the translation is correct.

image.png

Is it possible to add just a little more space above the collapsible menu?

Rob

Share this post


Link to post
Share on other sites
Ken Roy

Thanks Rob,

I found my problem.  I had the wrong $text variable in French. 

I am still studying the code but am not seeing how it is different from the toggle sections in the General Settings, other than it has collapsible sections for all settings, so I am not sure how I can add extra space above the toggle section.

Share this post


Link to post
Share on other sites
Ken Roy

I have posted v13.0.1.9a which adds German and Spanish translations and fixes the French section header displaying in English.

My attempts at adding white space above the toggle section were futile at this time.  I may pursue later but would like to get my own site upgraded to TNG 13.0.1

Share this post


Link to post
Share on other sites
Rob Severijns

No worries Ken,

It's just aesthetic so no need to rush.

Rob

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

×