Developer forum

Forum » Development » GetPageviewByPageID results in endless loop

GetPageviewByPageID results in endless loop

Vincent Gercke
Reply

We have recently upgraded a solution to 9.15.2 and since we can't use Dynamicweb.Frontend.PageView.Current() to access AreaSettings from the backend anymore, we tried to use GetPageviewByPageID instead as suggested by the annotation.

Unfortunately this results in a endless loop, which I suspect is due to the following change in the source code:

This section was moved to after was moved to after the PageLoaded notification is called:

This, as far as I can see, results in the PageView never being added to the cache and therefore Load keeps getting called and always treats the PageView as newly created (needs to be created and not fetched from the cache):

I hope it makes sense otherwise let me know and I will try to explain it again.

BR Vincent


Replies

 
Nicolai Pedersen Dynamicweb Employee
Nicolai Pedersen
Reply

Hi Vincent

We do not do bugs - so it must be you :-). (We will look at this one.

Why can you not do Pageview.Current().Area any more? What was your previous code? It should be compatible... So that might be another mistake.

 
Nuno Aguiar Dynamicweb Employee
Nuno Aguiar
Reply

Hi Nicolai,

 

Please check this other post https://doc.dynamicweb.com/forum?ThreadID=86182&PID=1605

 

Pageview.Current() is returning null

 

Best Regards,

Nuno Aguiar

 
Nicolai Pedersen Dynamicweb Employee
Nicolai Pedersen
Reply

Yikes!

Will dig into it.... Sorry!

 
Vincent Gercke
Reply

Thanks!

 
Nuno Aguiar Dynamicweb Employee
Nuno Aguiar
Reply

Hi Nicolai,

 

Is there a DevOps ID for this yet?

 

BR

Nuno

 
Morten Bengtson Dynamicweb Employee
Morten Bengtson
Reply

Bug #12311 has been registered in DevOps.
A fix will be provided ASAP.

/Morten 

 
Scott Forsyth Dynamicweb Employee
Scott Forsyth
Reply

Is there an estimated timeline on when the next release will come out? We watched for 9.15.3 this Tuesday (yesterday) but haven't seen it yet. Will there be a mid-week release, or is next Tuesday the current ETA?

Thanks,
Scott

 
Nicolai Pedersen Dynamicweb Employee
Nicolai Pedersen
Reply

Just take 9.15.5 of Dynamicweb:
https://www.nuget.org/packages/Dynamicweb/9.15.5

 
Scott Forsyth Dynamicweb Employee
Scott Forsyth
Reply

Good deal. I'll do that. Thanks!

 

You must be logged in to post in the forum