Allow translation of Asset titles

[robmcfadden]

Asset fields were recently added to the export: #8

This is great, but sometimes sites are using the title for the alt text instead of a field or for other reasons. If the title wasn't intentionally ignored for some reason, can we add that in?

https://github.com/robmcfadden/craft-translations/blob/asset-title-support/src/services/fieldtranslator/AssetsFieldTranslator.php#L40

Happy to make a PR for this. If this looks good, let me know which branch I should make the PR to. Thanks!

[ifrimere]

@robmcfadden - thanks for the ticket and more thanks for the PR offer.

@sidedwards is going to look into this and will get back to you with any questions, requested edits to the PR proposal and guidance about the branch where it can be placed against.

Thanks.

[sidedwards]

@robmcfadden great catch! We have existing logic to handle updating Asset titles from XML files, but must have missed it for file generation. Your proposed fix looks good and I've confirmed that it works as expected. Please create a new branch off of master called bug/asset-title-fix, and I can merge it in with our next release v1.10.5.

Thanks!

[robmcfadden]

@sidedwards Great, thanks! PR is going into master for now. Base branch can be changed to release/1.10.5 once available.

[sidedwards]

@robmcfadden merged #172 into release/1.10.5, thanks for the contribution!