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

Translated Comments from German to English #474

Conversation

arjun-kshirsagar
Copy link

Pull Request: Translation of German Comments to English

Overview

This pull request (PR) addresses the issue of German comments found in various source files within the LibreOffice repository. The primary goal is to translate these comments into English to enhance code readability and maintainability for all contributors.

Files Translated

The following files have been updated with translated comments:

  • core/src/main/java/org/libreoffice/lots/db/SchemaDatasource.java
  • core/src/main/java/org/libreoffice/lots/db/Search.java
  • core/src/main/java/org/libreoffice/lots/db/SearchStrategy.java
  • core/src/main/java/org/libreoffice/lots/db/SimpleDataset.java
  • core/src/main/java/org/libreoffice/lots/db/ThingyDatasource.java
  • core/src/main/java/org/libreoffice/lots/db/UnionDatasource.java

Summary of Changes

  • Translated German comments to English to improve understanding and accessibility for the international development team.
  • Ensured that the meaning and context of the original comments were preserved during translation.

Why This Change?

Translating comments to English is crucial for:

  • Improving code readability for international developers.
  • Ensuring consistency in documentation across the codebase.
  • Facilitating easier code maintenance and onboarding of new contributors.

Related Issues

@arjun-kshirsagar
Copy link
Author

arjun-kshirsagar commented Aug 4, 2024

I had changed the comments on the GitHub itself, so you can ignore the checks as no single line of code was changed apart from the German comments.

@thorstenb @smehrbrodt Please review the same and merge it.

Thanks.

Copy link

@laenion laenion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In general the quality of the translation is quite good (though I assume it is at least based on a machine translation?), so just adding a few minor comments.

* entferne alle anderen Sternchen. Ein Punkt am Ende eines Wortes wird als Abkürzung
* interpretiert und durch Sternchen ersetzt.
/**
* Only a single asterisk at the end of a word is accepted by the user. Therefore, remove all other asterisks.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is accepted by the user

I think this should be something like "is allowed as user input".

Comment on lines +181 to +183
* Takes a template for a search query (which may contain variables of the form "${suchanfrageX}")
* and instantiates it with words from words, considering only the first wordcount
* entries of words.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"suchanfrageX" should also be translated, such as "searchqueryX".

instantiates it with words from words

That's difficult to parse in English - in German you knew that the English term refers to the variable name, but in pure English that context gets lost. How about "instantiates it with a list of words"?

*/

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are several additional newlines in this pull request without any obvious pattern when they are added and when they are skipped...

* Note: The following condition is "unnecessarily" strict,
* but to relax it (e.g., the overall schema is the union of the schemas)
* it would be necessary to implement a dataset wrapper that ensures that
* all datasets returned in query results have the same schema.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

query results

This should be QueryResults.

@arjun-kshirsagar arjun-kshirsagar closed this by deleting the head repository Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants