phpbb bridging not working, fresh install

[Casper]

I had problems adding a phpbb bridge, using phpbb 2.0.11 and the current dev checkout.

It caused database error.  So I installed the latest 2.0.14 version of phpbb, and on another up to date 1.4 cpg install, had exactly the same problem.

The db error is as follows;

While executing query "SELECT user_id as user_id FROM `clive_cpg`.phpbb_users WHERE group_id in (1)" on 0

mySQL error: Unknown column 'group_id' in 'where clause'

On checking, I found there is no such thing as 'group_id' in the phpbb users table, the nearest thing is 'user_level'.  I tried various changes to the phpbb.inc.php, all without success, as it's a bit above me.

[Nibbler]

Caused by this code in init.inc.php

Code Select Expand

// Retrieve Array of Admin Users (used for hiding admin usernames on thumbnails)
$results = cpg_db_query("SELECT {$cpg_udb->field['user_id']} as user_id FROM $cpg_udb->usertable WHERE {$cpg_udb->field['usertbl_group_id']} in (" . implode(',',$CONFIG['ADMIN_GROUPS']).')');
$CONFIG['ADMIN_USERS']=array();
while ($row = mysql_fetch_array($results)) {
    $CONFIG['ADMIN_USERS'][] = $row['user_id'];
} // while
mysql_free_result($results);



You can't assume that all forums have such a simple user-group relationship, in fact most do not. I propose we cut this feature, and schedule it for 1.5 where it can be properly implemented. (unless someone has the time to find and test a solution for all 13 bbs).

[Nibbler]

Taking that silence as agreement, feature removed.

[Casper]

Just did an update, but still get the same error as originally posted;

While executing query "SELECT user_id as user_id FROM `clive_cpg`.phpbb_users WHERE group_id in (1)" on 0

mySQL error: Unknown column 'group_id' in 'where clause'

[Nibbler]

Try rev 1.92

[Casper]

Version 1.92 does not give an error, but admin is seen as a regular user, and gets the admin menus as a registered user, not admin, so cannot admin the gallery.

[Nibbler]

Unrelated issue. Marking this as fixed. Will commit fixed up phpbb bridge for this new issue shortly.

[donnoman]

That was an approved 1.4 feature.

• switch to show/hide admin username under thumbnails - see http://forum.coppermine-gallery.net/viewtopic.php?p=20779#20779  - done 2005-01-30 Donnoman
  

It should not have been removed without discussion.

[Nibbler]

There was discussion - http://forum.coppermine-gallery.net/index.php?topic=17541

[donnoman]

Sorry Nibbler, I missed this section as I do not use phpbb bridging so I never read this thread.

However you nixed a feature that I like for the sake of a single bridge that isn't working, I don't think thats a valid reason.

Fix either the bridging or the feature, I don't think nixing it is an acceptable solution.

[Nibbler]

The code you added breaks many of the bridges in order to add a trivial feature. I only commented out the code, it is still there if you can fix the bug.

[donnoman]

I don't understand where casper was having a problem with the original code.

Code Select Expand


// Retrieve Array of Admin Groups (used for hiding admin usernames on thumbnails)
$results = cpg_db_query("SELECT group_id FROM {$CONFIG['TABLE_USERGROUPS']} WHERE has_admin_access ");
$CONFIG['ADMIN_GROUPS']=array();
while ($row = mysql_fetch_array($results)) {
   $CONFIG['ADMIN_GROUPS'][]= $row['group_id'];
} // while
mysql_free_result($results);

// Retrieve Array of Admin Users (used for hiding admin usernames on thumbnails)
$results = cpg_db_query("SELECT {$cpg_udb->field['user_id']} as user_id FROM $cpg_udb->usertable WHERE {$cpg_udb->field['usertbl_group_id']} in (" . implode(',',$CONFIG['ADMIN_GROUPS']).')');
$CONFIG['ADMIN_USERS']=array();
while ($row = mysql_fetch_array($results)) {
   $CONFIG['ADMIN_USERS'][] = $row['user_id'];
} // while
mysql_free_result($results);


Just did an update, but still get the same error as originally posted;

While executing query "SELECT user_id as user_id FROM `clive_cpg`.phpbb_users WHERE group_id in (1)" on 0

mySQL error: Unknown column 'group_id' in 'where clause'

Which means theres an admin group with id of 1 set as administrators, but no users in the group... so how can any bridged user ADMIN his coppermine install?

He then reports:

Version 1.92 does not give an error, but admin is seen as a regular user, and gets the admin menus as a registered user, not admin, so cannot admin the gallery.

So this leads me to believe that if you correct the admin issue with the bridge, the original code for the admin uploader feature will work as designed.

[donnoman]

Nibbler what was the "unrelated" issue?

Casper can you re-enable all the oringinal admin uploader code and see if it works properly now that you have the admin problem fixed.

[Casper]

The admin problem is not fixed yet, so going back at the moment will just return me to the opening point of this thread.

I'm waiting for Nibblers fix to the phpbb.inc.php file.

[Nibbler]

It's how the tables are setup in the database of the forum.

Your query assumes the user table stores the group of the user. Coppermine does this, so the query works.

PHPBB and others use a usergroups table, so you have to join the 2 tables on user_id to get the groups for a specified user.

I don't think there is an easy fix here.

I already searched the whole code to check the impact of commenting out yours. The unrelated issue is unrelated, and the fix is not yet in cvs.

[donnoman]

Ahh sorry Casper, I didn't realise that.

Perhaps Nibbler can test the uploader code when he's finished with the admin fix.

[donnoman]

Nibbler, the group code is synced to coppermine, and thats how users can establish that they are the gallery admin period. Thats how coppermine works.

It's immaterial how the forum stores the information, the bridge HAS to give it to coppermine in that form, it has nothing to do with the uploader code.

The uploader code is just looking for the same thing that the rest of coppermine looks for to see if a user is "GALLERY_ADMIN".

Fix the admin problem, uploader code should work.

[Nibbler]

Commited fix, update phpbb bridge and udb_core. Ignore updated picmgmt.inc.php.

Don't try to tell me how coppermine works.

[donnoman]

Did you test the uploader code?

[Nibbler]

Check here