Skip to content

Conversation

girishsharma008
Copy link

Upgraded all sitecore librray related to Sitecore 10.4
and handled exception for below error -
Exception: System.NullReferenceException
Message: Object reference not set to an instance of an object. Source: Sitecore.Kernel
at Sitecore.Links.UrlBuilders.Helpers.ItemPathBuilder.GetRelativePath(Item item, SiteInfo site) at Sitecore.Links.UrlBuilders.Helpers.ItemPathBuilder.Build(Item item, SiteInfo site)

and handled exception for below error -
Exception: System.NullReferenceException
Message: Object reference not set to an instance of an object.
Source: Sitecore.Kernel
at Sitecore.Links.UrlBuilders.Helpers.ItemPathBuilder.GetRelativePath(Item item, SiteInfo site)
at Sitecore.Links.UrlBuilders.Helpers.ItemPathBuilder.Build(Item item, SiteInfo site)
removed unwanted folders
@muso31
Copy link
Collaborator

muso31 commented Jun 27, 2024

@girishsharma008 do you have access to the github actions to update these for a 10.4 release?

@girishsharma008
Copy link
Author

@girishsharma008 do you have access to the github actions to update these for a 10.4 release?

sorry I do not have any access. i have only sent PR so that owner's can merge if changes are ok.

@robmaas
Copy link
Contributor

robmaas commented Jun 28, 2024

@girishsharma008 The PR is missing changes in the Github workflow(s), probably should include that as well.

@girishsharma008 girishsharma008 marked this pull request as draft June 29, 2024 05:52
@girishsharma008 girishsharma008 marked this pull request as ready for review June 29, 2024 05:53
@girishsharma008 girishsharma008 marked this pull request as draft June 29, 2024 05:53
@girishsharma008 girishsharma008 marked this pull request as ready for review June 29, 2024 05:54
@girishsharma008
Copy link
Author

@girishsharma008 The PR is missing changes in the Github workflow(s), probably should include that as well.

@robmaas Thank you for letting me know i have done needful and pushed latest code


public static string GetItemUrl(Item item, UrlOptions urlOptions)
{
//Added check for when rendering paramater request comes as Sitecore.Links.UrlBuilders.Helpers.ItemPathBuilder.GetRelativePath
Copy link

Choose a reason for hiding this comment

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

Seems like a better backward compatible fix is to copy out the previous null check based on the notes from #463

Assert.True(true);
}
#endif
#if SC103
Copy link
Contributor

@robmaas robmaas Jul 1, 2024

Choose a reason for hiding this comment

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

Suggested change
#if SC103
#if SC104

(also wrong on rule 79, should be #if SC103 there)

@robmaas robmaas mentioned this pull request Jul 24, 2024
1 task
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.

4 participants