How do I properly extend a Shield Controller?

[EliMariaaaa]

Hello,

I'm trying to extend the RegisterController but I can't make it work.
I followed the doc and just extended the controller by creating app/Controllers/Registercontroller.php:

<?php

namespace App\Controllers;

use CodeIgniter\Shield\Controllers\RegisterController as ShieldRegister;
use CodeIgniter\HTTP\RedirectResponse;

class RegisterController extends ShieldRegister
{
    public function registerView()
    {
        return echo "test";
    }
    /**
     * Attempts to register the user.
     */
    public function registerAction(): RedirectResponse
    {
        if (auth()->loggedIn()) {
            return redirect()->to(config('Auth')->registerRedirect());
        }

        // Check if registration is allowed
        if (! setting('Auth.allowRegistration')) {
            return redirect()->back()->withInput()
                ->with('error', lang('Auth.registerDisabled'));
        }

        $users = $this->getUserProvider();

        // Validate here first, since some things,
        // like the password, can only be validated properly here.
        $rules = $this->getValidationRules();

        if (! $this->validateData($this->request->getPost(), $rules, [], config('Auth')->DBGroup)) {
            return redirect()->back()->withInput()->with('errors', $this->validator->getErrors());
        }
        print_r($this->request->getPost()); die();
        // Save the user
        $allowedPostFields = array_keys($rules);
        $user              = $this->getUserEntity();
        $user->fill($this->request->getPost($allowedPostFields));
        
        // Workaround for email only registration/login
        if ($user->username === null) {
            $user->username = null;
        }

        try {
            $users->save($user);
        } catch (ValidationException $e) {
            return redirect()->back()->withInput()->with('errors', $users->errors());
        }

        // To get the complete user object with ID, we need to get from the database
        $user = $users->findById($users->getInsertID());

        // Add to default group
        $users->addToDefaultGroup($user);

        Events::trigger('register', $user);

        /** @var Session $authenticator */
        $authenticator = auth('session')->getAuthenticator();

        $authenticator->startLogin($user);

        // If an action has been defined for register, start it up.
        $hasAction = $authenticator->startUpAction('register', $user);
        if ($hasAction) {
            return redirect()->route('auth-action-show');
        }

        // Set the user inactive
        $user->deactivate();//$user->activate();

        $authenticator->completeLogin($user);

        // Success!
        return redirect()->to(config('Auth')->registerRedirect())
            ->with('message', lang('Auth.registerSuccess'));
    }
}

I completely replaced what's inside the registerView() function but it's still showing the register form.
What am I missing?

[datamweb]

Hi,

What am I missing?

Because you have not set the routes for new controller. To do this you need to unset the default routes(service('auth')->routes($routes, ['except' => ['register']]);) from the circuit and then set up the routes with your new controller.
Read the document for more information.

[EliMariaaaa]

Hi @datamweb - thank you so much for your help! I just have a follow-up question. Do I need to re-declare everything that's needed to be used in the specific function I want to override? In my case, the one I'm trying to modify is the registerAction() function. When I try to register, I get this error:

Class 'App\Controllers\Auth\Events' not found

When I added use CodeIgniter\Events\Events; in the file, it works. My thinking is, I don't have to since this is just an extension of the original controller.

[datamweb]

When you want to use a class, you must import it. Here you have Events::trigger('register', $user); in function registerAction() so you need to import it by use CodeIgniter\Events\Events;.

For whatever reason, if you don't want to import the items in your controller, you can do it through App\Controllers\BaseController::initController(), which I don't recommend.

public function initController(RequestInterface $request, ResponseInterface $response, LoggerInterface $logger)
{
// Do Not Edit This Line
parent::initController($request, $response, $logger);
// Preload any models, libraries, etc, here.
+$this->events= new \CodeIgniter\Events\Events();
// E.g.: $this->session = \Config\Services::session();
}

-Events::trigger('register', $user);
+$this->events::trigger('register', $user);

But I was curious to know why you plan to use override. What exactly do you want to change?