Skip to content
Snippets Groups Projects
  1. Aug 14, 2020
  2. Aug 11, 2020
  3. Aug 10, 2020
  4. Jul 16, 2020
    • Timm Fitschen's avatar
      Merge branch 'dev' into f-versioning · a1599371
      Timm Fitschen authored
      Verified
      a1599371
    • Timm Fitschen's avatar
      Merge branch 'f-select-subproperties' into 'dev' · 30d3d3d4
      Timm Fitschen authored
      f-select-subproperties -> dev
      
      # Summary
      
      See summary of https://gitlab.com/caosdb/caosdb-webui/-/merge_requests/72
      
      Funny side note: this MR !47 and the issue it resolves #47 have the same number :grinning: 
      
      # Focus
      
      * The important changes are in `RetrieveFullEntity.java`, `EntityToElementStrategy.java` and `SetFieldStrategy.java`.
        * `RetrieveFullEntity.java` is responsible for the retrieval of entities and it needed additional functionality to retrieve referenced entities recursively. It does this but only if a list of `Query.Selects` is there and non-empty, because this is the case when the request is a SELECT query.
        * `EntityToElementStrategy.java` is responsible for the xml generation. It was kind of buggy and I did some refactoring there.
        * `SetFieldStrategy.java` is decides whether a property of an entity is to be included in the generated output (xml) or not. 
      Additional functionality is added to cope with deeply nested references.
      * Note: I also deleted the AWI stuff, because I didn't want to refactor these classes.
      
      # Test Environment
      
      See https://gitlab.com/caosdb/caosdb-webui/-/merge_requests/72
      
      # Check List for the Author
      
      - [x] All automated tests pass
      - [x] Reference related Issues
      - [x] Up-to-date CHANGELOG.md
      - [x] Annotations in code (Gitlab comments)
        - Intent of new code
        - Problems with old code
        - Why this implementation?
      
      
      # Check List for the Reviewer
      
      
      - [x] I understand the intent of this MR
      - [x] All automated tests pass
      - [x] Up-to-date CHANGELOG.md
      - [ ] The test environment setup works and the intended behavior is
        reproducible in the test environment
      - [x] In-code documentation and comments are up-to-date.
      - [ ] Check: Are there spezifications? Are they satisfied?
      
      For further good practices have a look at [our review guidelines](https://gitlab.com/caosdb/caosdb/-/blob/dev/REVIEW_GUIDELINES.md).
      
      See merge request caosdb/caosdb-server!47
      30d3d3d4
    • Timm Fitschen's avatar
  5. Jul 15, 2020
  6. Jul 13, 2020
  7. Jul 10, 2020
  8. Jul 09, 2020
  9. Jul 08, 2020
  10. Jul 07, 2020
  11. Jul 06, 2020
  12. Jul 05, 2020
  13. Jul 03, 2020
  14. Jul 02, 2020
  15. Jun 30, 2020
  16. Jun 25, 2020
  17. Jun 22, 2020
Loading