Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fixing issue when trying to export datarow for list instances #531

Merged
merged 1 commit into from
Dec 2, 2021

Conversation

magarma
Copy link
Contributor

@magarma magarma commented Nov 24, 2021

There is an issue in the code that is preventing to export properties for files from libraries when there are several content types.
Not sure if affects to more cases.
In line 587, code is adding ContentTypeId to the pnpFile.Properties. In line 613, was trying to add again the same property key to the dictionary, that was raising an issue.
Changing the way to add properties to the dictionary, it will overwrite it if exists.
image

There is an issue in the code that is preventing to export properties for files from libraries when there are several content types.
Not sure if affects to more cases. 
In line 587, code is adding ContentTypeId to the pnpFile.Properties. In line 613, was trying to add again the same property key to the dictionary, that was raising an issue.
Changing the way to add properties to the dictionary, it will overwrite it if exists.
@jansenbe jansenbe self-assigned this Dec 1, 2021
@jansenbe
Copy link
Contributor

jansenbe commented Dec 2, 2021

Good change @magarma , thanks for helping make PnP Provisioning better 💗

jansenbe added a commit that referenced this pull request Dec 2, 2021
@jansenbe jansenbe merged commit 178d66b into pnp:dev Dec 2, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants