Skip to content
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

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

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

@@ -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
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

@@ -81,6 +81,12 @@ public void Sc103()
{
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