Exporting to google spreadsheet

More
3 years 4 months ago #1470 by SimonAL
Hi,

Using 2.6.5 f5

When Exporting data to Google SS i relized that some Terms were mixed up and corrupted.
I had a lot of groups, some of them were: Building, BuildingUI, BuildingUpgrades...

The terms in them were:
Building/Something
BuildingUI/SomethingElse
BuildingUpgrade/SomethingReallyDifferent

In the spredsheet i got terms like:
Something
I/SomethingElse
pgrade/SomethingReallyDifferent

Easy to see now but hard to spot them between 2000 terms ;)

After some debugging I found out that you check the terms assigned to a group like this (LanguageSource_Export_CSV.cs):
if (string.IsNullOrEmpty(Category) || (Category==EmptyCategory && termData.Term.IndexOfAny(CategorySeparators)<0))
        Term = termData.Term;
else
	if (termData.Term.StartsWith(Category) && Category!=termData.Term)
		Term = termData.Term.Substring(Category.Length+1);
else
	continue;	// Term doesn't belong to this category

By using this check: if (termData.Term.StartsWith(Category) && Category!=termData.Term)
a mess happens because some groups starts with the same string... and we can see where the bug comes from...

I changed the script to: if (termData.Term.StartsWith(Category + @"/") && Category!=termData.Term)
and it works ok for now but I think this should be solved internally :)

Hope this helps. :blink:

Please Log in or Create an account to join the conversation.

More
3 years 4 months ago #1475 by Frank
Good Catch!
I will add a fix for this issue!

Thanks a lot for looking into it!
Frank

Are you :-) Give I2L 5 stars!
Are you :-( Please lets us know how to improve it!
To get the betas as soon as they are ready, check this out

Please Log in or Create an account to join the conversation.

More
3 years 4 months ago #1477 by Frank
This fix plus a few others are now in 2.6.6a4 which can be downloaded from the beta folder.

Thanks,
Frank

Are you :-) Give I2L 5 stars!
Are you :-( Please lets us know how to improve it!
To get the betas as soon as they are ready, check this out

Please Log in or Create an account to join the conversation.

Time to create page: 0.149 seconds
Template by JoomlaShine