(2018-07-05, 11:05 AM)Wires Wrote: I don’t believe this required a notification for developers as it instructs you about template changes here. We can lead you to the page but we cannot force you to read information. If you read point 3 in “Beginning the Upgrade” you wouldn’t of had such an issue. I will look back over the portal page when upgrading and see what adjustments are needed.
Thanks for the report.
With heavier themes such as NetPen comparing difs isn't going to work as they are soo changed a diff report will pick up nearly every file.
Just try it on a theme like NetPen or BootBB which I just did and 50+ changed files came up.
If there is a critical change in how the current theme and templating system works like this with how logins are handled in the template etc that needs to be notified and addressed with along with the change notes.
You can't have a blanket statement of compare difs and then expect critical forum changing changes to be found when heavily modified themes use 50+ changed files from the default theme.
and as for your point in question
Quote:If the blog post announcement mentions that there are theme changes, then perform the following:
- Go to Admin CP > Templates & Style > Templates > Find Updated Templates. This will show you a list of all the templates that have changed during the upgrade.
- You can either revert these templates to their default - meaning all the changes you’ve made to it will be removed - or you can see a Diff Report which will show you exactly what’s changed. If you have a custom theme installed, it is probably best that you look at the Diff Report and apply the changes you need.
Note if the blog post mentioned theme changes, this change or requirement in the login wasn't mentioned.
The only thing mentioned was:
Quote:If you use the login_attempt_check() function, note that its signature has changed.
If you go to that github post it is about the 'loginlockoutexpiry' code not the member login code.
Plus nowhere in the blog or that github page mentions this changed line I mentioned above.
<input name="my_post_key" type="hidden" value="{$mybb->post_code}" />
So my post still stands it should have been QA mentioned.