Skip to content

Ability to view string and ID cache value#75

Open
Antonytm wants to merge 1 commit into
Sitecore:masterfrom
Antonytm:master
Open

Ability to view string and ID cache value#75
Antonytm wants to merge 1 commit into
Sitecore:masterfrom
Antonytm:master

Conversation

@Antonytm
Copy link
Copy Markdown

Code provides ability to view values saved in cache. It uses reflection to get cache entry value for caches(due to internal and private access modifiers). It provides ability to see what is inside HtmlCache and PathsCache. This code could be easily extended to add more abilities to view what is inside other caches. All that is required: 1) Add condition for type of current cache entry 2) Add code that will convert current object to string value.

Examples:
image
image

Code provides ability to view values saved in cache. It uses reflection to get cache entry value for caches(due to internal and private access modifiers). It provides ability to see what is inside HtmlCache and PathsCache. This code could be easily extended to add more abilities to view what is inside other caches. All that is required: 1) Add condition for type of current cache entry 2) Add code that will convert current object to string value.
@DerDreschner
Copy link
Copy Markdown

I would really like to see that feature in sitecore.rocks

@Antonytm
Copy link
Copy Markdown
Author

Any particular reason why this pull request is ignored? :-(

@nickwesselman
Copy link
Copy Markdown
Contributor

Hi @Antonytm,

Sorry, not ignored. I took over Rocks about a year ago and we have not really had the capacity to do anything beyond maintenance on it / updates for new Sitecore and VS versions. This looks like a potentially useful feature but, to be honest, I am holding off on any new features for Rocks until I can get some telemetry on usage of existing features and some more test automation in place. Taking this PR for example... it uses reflection on internal/private members of a Caching API that I know was refactored in 8.2, so this would need to be tested across several Sitecore versions. There are lots of features in Rocks that need this and at the moment I'm not sure how much any of them are being used by the community.

Keeping the PR open though as I know it could be a useful feature. Thanks for the patience.

-Nick

@DerDreschner
Copy link
Copy Markdown

DerDreschner commented Jul 11, 2019

Hey @nickwesselman,

thank you very much for your reply! You're right, it definitely need some more testing and fixing. For example, I get this error for several but not all caches with Sitecore 8.2.7:

image

As the main feature isn't merged into master yet, I guess this is the right place to report it.

Thank you very much!

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.

3 participants