Lang Export From Grids

[Phil]

Hi guys,

we have a nice export tool, where we are able to define specific labels to export, as well as a nice grid for labels, where we can filter them using top grid filters.

But when it comes to export a specific list, such as the result of a filtering, then there's no other solution than copy labels name manually, and then paste them into export window dedicated area (and spend a lot of time on this if it's about 100 labels).

I suggest to have a new toolbar button, where selected labels in grid view would fill the export window which would appear when pressing on it. I suggest the same for email events of course. This export should contains ONLY selected labels or events (actually, we can't get rid of currency list, as well as events, at least in 5.1.x).

What do you think?

p

[Alexander Obuhovich]

Sounds interesting. I suppose only labels from language selected in labels list should be exported?

--
Best Regards,

http://www.in-portal.com
http://www.alex-time.com

[Phil]

sounds the best. And thanks to copy/pasting label list in export window, you can easily keep a copy to re-use for exporting another lang :)

Envoyé avec Sparrow

[Alexander Obuhovich]

You thought that label names only got exported? What I meant is that result of Export button pressing will be same XML file (language pack), that would be produced by:

1. filter needed phrases in grid
2. copy their labels
3. open "English" (language selected in dropdown directly above label list) export dialog
4. paste labels copied before
5. hitting Export button

[Dmitry A.]

Yeah, I like the idea too.

It would convenient to have the ability to export only what you have Filtered out or clicked on right from Phrases / Labels grid.

DA

[Phil]

My idea is not to copy/paste, but to have this export button while viewing labels OR events, which would export ONLY selected labels OR events.

As you can guess, this idea came from a personal need, but it seems very useful to anyone who is using in-portal in daily usage... like me :)

Envoyé avec Sparrow

[Alexander Obuhovich]

Then I guess we're ready for a task.

[Dmitry A.]

Here is the task:

1196: Ability to Export phrases from Labels & Phrases section

I have put it for 5.2.0 - it would be nice to have it there. Do you think it's possible?

DA

[Alexander Obuhovich]

Yes, of course.

[Phil]

Dmitry, thanks for task creation. You've missed to talk about Events export, just for notice...

Envoyé avec Sparrow

[Dmitry A.]

Yes, this is true - we might want to have the same for Events or something.

Alex what do you think?

DA

[Alexander Obuhovich]

That was the idea in original post. We can update existing task for sure, since at the end same export code (LanguageExportHelper) will be used.

[Phil]

while we are mopdifying the task, I'd like to mention here that we always have "countries" XML output in lang packs. I think this should disappear as long as we use selective export.

Envoyé avec Sparrow

[Alexander Obuhovich]

I don't know about that. Maybe we rather add an option (checkbox) named "Export Countries" which will be enabled by default.

We actually plan to move currency translations into language pack too, but currencies are located in In-Commerce.

I don't know exactly yet, how to process that correctly.

[Phil]

why not a checkbox. The goal is while you are working on export stuff, to take the opportunity to make it working well completely :-)

Envoyé avec Sparrow

[Dmitry A.]

Hi guys,

Here is some more ideas that we can do in this task to make things more logical:

1. We add ability to select Data Type to Export - checkboxes on Export screen for Lang Export via Regional section (all checked by default) - see 1st screenshot

2. On Phrases & Labels and Events we only show similar Export screen, BUT remove unneeded fields - see 2nd screenshot. 

What do you think?

DA

[Alexander Obuhovich]

Looks good.

[Dmitry A.]

Great - I have updated the task so it fully reflects what we have here.

Note that I had to rename the task so here it is:

Improvements to Languages and Phrases Export capabilities

http://tracker.in-portal.org/view.php?id=1196 

DA

[Phil]

Dmitry, for step 2, need also to create a new export button in grid, when viewing labels and events :)

Envoyé avec Sparrow

[Dmitry A.]

Thanks for picking this up Phil.

This is exactly as I described it in the task itself - "Export" toolbar buttons should HIDDEN for Simple system preset :)

DA

[Phil]

hehe, I just worried that you said it should be hidden, but didn't said where it should appear otherwise... Ok, it's maybe obvious :)

Envoyé avec Sparrow

[Dmitry A.]

It would be great if we can add ModifiedOn field to Email Events since currently we don't have it.

The field exists for List of Phrases and let's you see the most updated/changed so you can export only them.

Anyone against of adding new field as part of this task?

DA

[Phil]

+1 :)

Envoyé avec Sparrow

[Alexander Obuhovich]

Done. Ready for testing.

[Phil]

great !

Envoyé avec Sparrow

Pièces jointes :

- direct_export_from_phrase_email_event_grid.patch

[Dmitry A.]

Hi guys,

Looks like there is a Bug in task related to this discussion.

After recent improments to Language Pack Export/Import functionality described in following to tasks - we have lost ability to Export Body of Email Events.

1227: Ability to specify both HTML and Plain Text version of e-mail event - http://tracker.in-portal.org/view.php?id=1227 

1224: Email event usage refactoring - http://tracker.in-portal.org/view.php?id=1224 

See attachment with exported Language pack which is missing Body of Email Events (Subject is there).

Please correct me if I am wrong. I guess we should create separate tasks since 5.2.0-B3 already has been released.

DA

[Phil]

nice catch, but don't it miss also message type parameter and value in <EVENT>? Or have it changed recently?



Envoyé avec Sparrow (http://www.sparrowmailapp.com/?sig)

> Pièces jointes :
> - TestEnglishExport.lang
>

[Alexander Obuhovich]

Type parameter is there, e.g. <EVENT Event="USER.VALIDATE" Type="0">

[Phil]

sorry, I mispelled it, I wanted to say "MessageType" parameter :)

Envoyé avec Sparrow (http://www.sparrowmailapp.com/?sig)

[Alexander Obuhovich]

That's what refactoring in http://tracker.in-portal.org/view.php?id=1227 was all about: allow to specify both html and plain-text versions of e-mail event at same type and not only one of them + MessageType attribute to indicate which one exactly.

[Alexander Obuhovich]

Dmitry, please provide me exact export dialog parameters you've used.

I've tried both direct export from "E-mail Events" section and from "Regional" section and I've always had <HTMLBODY> node in xml present.

From the code I can tell, that the only case, when both PLAINTEXTBODY and HTMLBODY nodes are missing in export is when you export non-primary language that has no translation in these e-mail events.

[Dmitry A.]

Sorry Alex - it does work just fine.

I have had 5.2.0 reinstalled to make sure DB structure is right.

Issue dismissed.

DA