-
Notifications
You must be signed in to change notification settings - Fork 121
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
Upgraded all sitecore librray related to Sitecore 10.4 #465
base: master
Are you sure you want to change the base?
Upgraded all sitecore librray related to Sitecore 10.4 #465
Conversation
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)
This reverts commit 04e94b0.
removed unwanted folders
@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. |
@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 |
@@ -35,6 +35,13 @@ public static string GetMediaUrl(MediaItem media) | |||
|
|||
public static string GetItemUrl(Item item, UrlOptions urlOptions) | |||
{ | |||
//Added check for when rendering paramater request comes as Sitecore.Links.UrlBuilders.Helpers.ItemPathBuilder.GetRelativePath |
There was a problem hiding this comment.
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
@@ -81,6 +81,12 @@ public void Sc103() | |||
{ | |||
Assert.True(true); | |||
} | |||
#endif | |||
#if SC103 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#if SC103 | |
#if SC104 | |
(also wrong on rule 79, should be #if SC103
there)
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)