Skip to content

Conversation

stevejgordon
Copy link
Contributor

Fixes #7075.

@stevejgordon stevejgordon added backport 8.5 8.x Relates to a 8.x client version labels Dec 6, 2022
@stevejgordon stevejgordon changed the title Fix dictionary properties Fix resolvable dictionary properties Dec 6, 2022
@stevejgordon stevejgordon merged commit 5b78d56 into main Dec 6, 2022
@stevejgordon stevejgordon deleted the bug-7075 branch December 6, 2022 15:17
github-actions bot pushed a commit that referenced this pull request Dec 6, 2022
* Prefer IReadOnlyDictionary on read only (response) types

* Introduce readonly dictionary converters for Field and IndexName

These ensure keys are santised during lookups so that items can be found correctly using inferred values.

* Prefer IDictionary for properties on writable types

* Update tests

* Fix BOM

* Cleanup IUrlParameter types for consistency
github-actions bot pushed a commit that referenced this pull request Dec 6, 2022
* Prefer IReadOnlyDictionary on read only (response) types

* Introduce readonly dictionary converters for Field and IndexName

These ensure keys are santised during lookups so that items can be found correctly using inferred values.

* Prefer IDictionary for properties on writable types

* Update tests

* Fix BOM

* Cleanup IUrlParameter types for consistency
github-actions bot pushed a commit that referenced this pull request Dec 6, 2022
* Prefer IReadOnlyDictionary on read only (response) types

* Introduce readonly dictionary converters for Field and IndexName

These ensure keys are santised during lookups so that items can be found correctly using inferred values.

* Prefer IDictionary for properties on writable types

* Update tests

* Fix BOM

* Cleanup IUrlParameter types for consistency
github-actions bot pushed a commit that referenced this pull request Dec 6, 2022
* Prefer IReadOnlyDictionary on read only (response) types

* Introduce readonly dictionary converters for Field and IndexName

These ensure keys are santised during lookups so that items can be found correctly using inferred values.

* Prefer IDictionary for properties on writable types

* Update tests

* Fix BOM

* Cleanup IUrlParameter types for consistency
github-actions bot pushed a commit that referenced this pull request Dec 6, 2022
* Prefer IReadOnlyDictionary on read only (response) types

* Introduce readonly dictionary converters for Field and IndexName

These ensure keys are santised during lookups so that items can be found correctly using inferred values.

* Prefer IDictionary for properties on writable types

* Update tests

* Fix BOM

* Cleanup IUrlParameter types for consistency
github-actions bot pushed a commit that referenced this pull request Dec 6, 2022
* Prefer IReadOnlyDictionary on read only (response) types

* Introduce readonly dictionary converters for Field and IndexName

These ensure keys are santised during lookups so that items can be found correctly using inferred values.

* Prefer IDictionary for properties on writable types

* Update tests

* Fix BOM

* Cleanup IUrlParameter types for consistency
github-actions bot pushed a commit that referenced this pull request Dec 6, 2022
* Prefer IReadOnlyDictionary on read only (response) types

* Introduce readonly dictionary converters for Field and IndexName

These ensure keys are santised during lookups so that items can be found correctly using inferred values.

* Prefer IDictionary for properties on writable types

* Update tests

* Fix BOM

* Cleanup IUrlParameter types for consistency
stevejgordon added a commit that referenced this pull request Dec 6, 2022
* Prefer IReadOnlyDictionary on read only (response) types

* Introduce readonly dictionary converters for Field and IndexName

These ensure keys are santised during lookups so that items can be found correctly using inferred values.

* Prefer IDictionary for properties on writable types

* Update tests

* Fix BOM

* Cleanup IUrlParameter types for consistency

Co-authored-by: Steve Gordon <sgordon@hotmail.co.uk>
stevejgordon added a commit that referenced this pull request Dec 6, 2022
* Prefer IReadOnlyDictionary on read only (response) types

* Introduce readonly dictionary converters for Field and IndexName

These ensure keys are santised during lookups so that items can be found correctly using inferred values.

* Prefer IDictionary for properties on writable types

* Update tests

* Fix BOM

* Cleanup IUrlParameter types for consistency

Co-authored-by: Steve Gordon <sgordon@hotmail.co.uk>
stevejgordon added a commit that referenced this pull request Dec 6, 2022
* Prefer IReadOnlyDictionary on read only (response) types

* Introduce readonly dictionary converters for Field and IndexName

These ensure keys are santised during lookups so that items can be found correctly using inferred values.

* Prefer IDictionary for properties on writable types

* Update tests

* Fix BOM

* Cleanup IUrlParameter types for consistency

Co-authored-by: Steve Gordon <sgordon@hotmail.co.uk>
stevejgordon added a commit that referenced this pull request Dec 6, 2022
* Prefer IReadOnlyDictionary on read only (response) types

* Introduce readonly dictionary converters for Field and IndexName

These ensure keys are santised during lookups so that items can be found correctly using inferred values.

* Prefer IDictionary for properties on writable types

* Update tests

* Fix BOM

* Cleanup IUrlParameter types for consistency

Co-authored-by: Steve Gordon <sgordon@hotmail.co.uk>
stevejgordon added a commit that referenced this pull request Dec 6, 2022
* Prefer IReadOnlyDictionary on read only (response) types

* Introduce readonly dictionary converters for Field and IndexName

These ensure keys are santised during lookups so that items can be found correctly using inferred values.

* Prefer IDictionary for properties on writable types

* Update tests

* Fix BOM

* Cleanup IUrlParameter types for consistency

Co-authored-by: Steve Gordon <sgordon@hotmail.co.uk>
stevejgordon added a commit that referenced this pull request Dec 6, 2022
* Prefer IReadOnlyDictionary on read only (response) types

* Introduce readonly dictionary converters for Field and IndexName

These ensure keys are santised during lookups so that items can be found correctly using inferred values.

* Prefer IDictionary for properties on writable types

* Update tests

* Fix BOM

* Cleanup IUrlParameter types for consistency

Co-authored-by: Steve Gordon <sgordon@hotmail.co.uk>
stevejgordon added a commit that referenced this pull request Dec 6, 2022
* Prefer IReadOnlyDictionary on read only (response) types

* Introduce readonly dictionary converters for Field and IndexName

These ensure keys are santised during lookups so that items can be found correctly using inferred values.

* Prefer IDictionary for properties on writable types

* Update tests

* Fix BOM

* Cleanup IUrlParameter types for consistency

Co-authored-by: Steve Gordon <sgordon@hotmail.co.uk>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.x Relates to a 8.x client version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dictionary properties which have IUrlParameters types as keys should have ResolvableReadOnlyDictionaryConverter applied
1 participant