XF2 [8WR] Discord Integration

XF2 [8WR] Discord Integration [Paid] 2.3.0.4

No permission to buy ($30.00)
Overview FAQ Updates (40) Reviews (39) History Discussion
Jaxel updated XF2 [8WR] Discord Integration with a new update entry:

2.1.1.1 - CHANGELOG

This is a continuation of the 2.1.1.0 update that came out a few hours ago. I HIGHLY recommend you read the patch notes for that update as well:

xenforo.com

XF2 [8WR] Discord Integration - Updates

xenforo.com xenforo.com

This update continues with the normalization and standardization which began in the last update. However, this update is a bit trickier. In this version, the connected account provider has been changed from...
Click to expand...

Read the rest of this update entry...
 
Unfortunately, we can't just rename connected accounts and call it a day.
Click to expand...
You can, actually :)

I took the liberty of writing the upgrade method for you:
PHP: 
    public function upgrade2111Step1(array $stepParams)
    {
        $position = empty($stepParams[0]) ? 0 : $stepParams[0];
        $perPage = 250;
     
        $db = $this->db();
     
        if (!isset($stepParams['max']))
        {
            $stepParams['max'] = $db->fetchOne("
                SELECT MAX(user_id)
                FROM xf_user_connected_account
                WHERE provider = 'discord'
            ");
        }
     
        $userIds = $db->fetchAllColumn($db->limit(
            "
                SELECT DISTINCT user_id
                FROM xf_user_connected_account
                WHERE user_id > ?
                    AND provider = 'discord'
                ORDER BY user_id
            ", $perPage
        ), $position);
        if (!$userIds)
        {
            return true;
        }
     
        $db->beginTransaction();

        $repo = \XF::repository('XF:ConnectedAccount');
     
        $connectedAccounts = \XF::finder('XF:UserConnectedAccount')->where('user_id', $userIds)->with(['User'])->fetch();
        foreach ($connectedAccounts as $connectedAccount)
        {
            $connectedAccount->fastUpdate('provider', 'ewr_discord');
            $repo->rebuildUserConnectedAccountCache($connectedAccount->User);
        }
     
        $db->commit();
     
        $next = end($userIds);
     
        return [
            $next,
            "{$next} / {$stepParams['max']}",
            $stepParams
        ];
    }

This is untested, but I don't see why it wouldn't work. If it comes down to it, I'll test it on the live site running this add-on as that site certainly cannot afford to lose all associations.

EDIT: Made some tweaks based on further reading of the code
 
Last edited:
Jaxel said:
Jaxel updated XF2 [8WR] Discord Integration with a new update entry:
Click to expand...
Unfortunately, we can't just rename connected accounts and call it a day. ALL users will be required to reconnect their discord accounts to their forum accounts. Previous connected accounts with Discord will be deleted with this upgrade in order to prevent email clashes as well. That's just the way it is. Not much that can be done about it.
Click to expand...

Deleting member accounts is not okay. Please consider changing this update so that it does not delete member accounts.

Destroying existing links is also not okay. It took me months to get everyone properly linked up on my forum - forcing everyone to re-link is a big problem too.

Please re-consider this update. There has to be a way to be compliant with xenoforo guidelines without deleting member accounts or forcing everyone to re-link.
 
Last edited:
DragonByte Tech said:
You can, actually :)

I took the liberty of writing the upgrade method for you:
PHP: 
    public function upgrade2111Step1(array $stepParams)
    {
        $position = empty($stepParams[0]) ? 0 : $stepParams[0];
        $perPage = 250;
    
        $db = $this->db();
    
        if (!isset($stepParams['max']))
        {
            $stepParams['max'] = $db->fetchOne("
                SELECT MAX(user_id)
                FROM xf_user_connected_account
                WHERE provider = 'discord'
            ");
        }
    
        $userIds = $db->fetchAllColumn($db->limit(
            "
                SELECT DISTINCT user_id
                FROM xf_user_connected_account
                WHERE user_id > ?
                    AND provider = 'discord'
                ORDER BY user_id
            ", $perPage
        ), $position);
        if (!$userIds)
        {
            return true;
        }
    
        $db->beginTransaction();

        $repo = \XF::repository('XF:ConnectedAccount');
    
        $connectedAccounts = \XF::finder('XF:UserConnectedAccount')->where('user_id', $userIds)->with(['User'])->fetch();
        foreach ($connectedAccounts as $connectedAccount)
        {
            $connectedAccount->fastUpdate('provider', 'ewr_discord');
            $repo->rebuildUserConnectedAccountCache($connectedAccount->User);
        }
    
        $db->commit();
    
        $next = end($userIds);
    
        return [
            $next,
            "{$next} / {$stepParams['max']}",
            $stepParams
        ];
    }

This is untested, but I don't see why it wouldn't work. If it comes down to it, I'll test it on the live site running this add-on as that site certainly cannot afford to lose all associations.

EDIT: Made some tweaks based on further reading of the code
Click to expand...
Okay, I've replaced the upgrade step with your code.
 
DragonByte Tech said:
You can, actually :)
Click to expand...
I actually had to make slight tweaks to your code. Your code only worked on forums which only used Discord. If they used facebook, twitter, google, etc as well, it had issues.
Code: 
    public function upgrade2111Step1(array $stepParams)
    {
        $position = empty($stepParams[0]) ? 0 : $stepParams[0];
        $perPage = 250;
    
        $db = $this->db();
    
        if (!isset($stepParams['max']))
        {
            $stepParams['max'] = $db->fetchOne("
                SELECT MAX(user_id)
                FROM xf_user_connected_account
                WHERE provider = 'discord'
            ");
        }
    
        $userIds = $db->fetchAllColumn($db->limit("
                SELECT DISTINCT user_id
                FROM xf_user_connected_account
                WHERE user_id > ?
                    AND provider = 'discord'
                ORDER BY user_id
            ", $perPage
        ), $position);
        
        if (!$userIds) { return true; }
    
        $db->beginTransaction();
    
        $connectedAccounts = \XF::finder('XF:UserConnectedAccount')->where([
            ['user_id', $userIds],
            ['provider', 'discord']
        ])->with(['User'])->fetch();
        
        $repo = \XF::repository('XF:ConnectedAccount');
        foreach ($connectedAccounts as $connectedAccount)
        {
            $connectedAccount->fastUpdate('provider', 'ewr_discord');
            $repo->rebuildUserConnectedAccountCache($connectedAccount->User);
        }
        
        $db->commit();
    
        $next = end($userIds);
        return [$next, "{$next} / {$stepParams['max']}", $stepParams];
    }
 
Wandernought said:
Do you plan to also fix it so it won't delete our member accounts?
We can't really upgrade while there's a risk that member accounts will be deleted by the upgrade.
Click to expand...
That's what the current 2.1.1.1 does. Download the most recent version.
 
Jaxel said:
That's what the current 2.1.1.1 does. Download the most recent version.
Click to expand...
I used the update link you emailed us.
It resulted in a broken install. Not only was every linked user delinked, but the link page did not even work correctly. It displayed buttons with broken text like "con_acc_desc.discord"to users.
 
Last edited:
I am on 2.1.1.1, opening the Discord Log within the Admin panel gives me this:

ErrorException: [E_WARNING] Invalid argument supplied for foreach() in src/addons/EWR/Discord/Repository/Log.php at line 36

XF::handlePhpError() in src/addons/EWR/Discord/Repository/Log.php at line 36
EWR\Discord\Repository\Log->groupLogs() in src/addons/EWR/Discord/Admin/Controller/Log.php at line 50
EWR\Discord\Admin\Controller\Log->actionDiscord() in src/XF/Mvc/Dispatcher.php at line 350
XF\Mvc\Dispatcher->dispatchClass() in src/XF/Mvc/Dispatcher.php at line 257
XF\Mvc\Dispatcher->dispatchFromMatch() in src/XF/Mvc/Dispatcher.php at line 113
XF\Mvc\Dispatcher->dispatchLoop() in src/XF/Mvc/Dispatcher.php at line 55
XF\Mvc\Dispatcher->run() in src/XF/App.php at line 2184
XF\App->run() in src/XF.php at line 391
XF::runApp() in admin.php at line 13
Click to expand...
 
I just upgraded and don't have the option to link Discord in connected accounts..

1580235260479.webp

Provider options are blank

1580235274464.webp

How do I fix this?
 
Did you guys run the upgrade script? It should be looking for con_acc.ewr_discord, not con_acc.discord. Try rebuilding the addon.
 
Can you go into your DB, and into the table xf_connected_account_provider. Rename discord to ewr_discord and tell me if that fixess it.

I forgot to put that line in the upgrade script.
 
Hello,

After the update, it appears that my bot isn't sending private messages to people to notify them of forum conversations. I didn't notice anything regarding that on the change log. No error logs for the add-on. I confirmed that all token and keys are correct for the bot.

The bot still posts when a thread/etc is made in a forum, but its just not sending private messages as it used to.

As a note, running recommended PHP etc.

Any help is appreciated. Thanks!
 
You must log in or register to reply here.

Similar threads

Umut12
  • Locked
Not a bug [8WR] Discord Integration 2.1.1.7 bug
Replies
1
Views
615
Chris D
Chris D
Naz
Discord Integration [Paid]
24 25 26
Replies
504
Views
37K
Wing
Wing
Jaxel
XF2 [8WR] XenCarta 2 (Wiki) PRO [Paid]
21 22 23
Replies
459
Views
38K
philmckrackon
philmckrackon
Jaxel
XF2 [8WR] XenMedio 2 (Media) PRO [Paid]
18 19 20
Replies
384
Views
26K
🔥Iggy🔥
🔥
Jaxel
XF2 [8WR] XenTorneo 2 (Rankings) PRO [Paid]
2 3 4
Replies
65
Views
6K
Jaxel
Jaxel
Back
Top Bottom