Force Custom User Field Completion after Registration or Login

Force Custom User Field Completion after Registration or Login [Paid] v1.9.0

No permission to buy ($15.00)
This add-on breaks all external CSS and JS files.

It's due to the line:

Code:
$controller = \XF::app()->router()->routeToController(\XF::app()->request()->getRoutePath())->getController();

Because when you load css.php - \XF::app()->request()->getRoutePath() will return blank, which then fed into routeToController returns XF:Index.

This means the CSS file passes the controller check and then returns a redirect to account/account-details

I thought a quick bodge to skip quit out the function if the URI was css.php, but that's no good as there are many other possible files it could be that would all be affected.

@Wutime - this is a showstopper :(
 
Last edited:
Because when you load css.php - \XF::app()->request()->getRoutePath() will return blank, which then fed into routeToController returns XF:Index.

This means the CSS file passes the controller check and then returns a redirect to account/account-details
This would suggest that pages would be loading without CSS in places, but that's not the case. They're routed to complete their details and forced to do so, only if a session is present and all conditions are met, otherwise it just returns the session untouched.

Working on updates this week so I'm open to a conversation, and thanks for your insight.
 
Wutime updated Force Custom User Field Completion after Registration or Login with a new update entry:

v1.9.0 - XF 2.3 Compliant - Forwards/Backwards Compatible

** XF 2.3 Compliant - Forwards/Backwards Compatible **

XF 2.3
  • Resolved issues with XF 2.3 (redirections and fails)
  • Resolved issue with admins being forced from admin.php
  • Extended _preSave on UserProfile and integrated with verify fields system where possible
    • System now prompts with overlay similar to default fields for most fields
  • Attempts to clear fields that are required fail
  • Avatar required will still require additional page load (no change)...

Read the rest of this update entry...
 
Back
Top Bottom