IntegraMOD / IntegraMOD151

The most complete premodded forum software
http://integramod.com
4 stars 4 forks source link

Nick Coloring #56

Closed vendethiel closed 5 years ago

vendethiel commented 6 years ago

Can someone give me a quick rundown, of how username coloring works in IntegraMOD? It doesn't seem to be integrated with e.g. the chatbox.

Are we able to customize it?

vendethiel commented 6 years ago

It seems like there's no way to change that, I'm really surprised.

The one I used was RCS (Rank Color System) but it's obviously woefully inadequate with PCP...

MWE001 commented 6 years ago

You know now that you mention that, some time back, I do believe you are right. I don't ever remember that ever being a thing in Integramod in the Shoutbox. Not sure why. Surely it wasn't a query deal was it like the flags all over the place was?

vendethiel commented 6 years ago

No, I added a single column to the shoutbox query and it works. It doesn’t have 1 query per chatter

MWE001 commented 6 years ago

Nice! That works. I'm almost thinking Teelk's advanced group color mod originally took care of that. Nice fix!

vendethiel commented 6 years ago

Well, ultimately I need to pick a mod to add group colors. I think it's a feature people really expect to have. Do you have a link to Teelk's?

MWE001 commented 6 years ago

Teelk's mod added colors to everything everywhere in every fashion. It was very at length and took a very long time to install. It was always the very first thing to ever get installed on IM 1,4,1 for a very good reason.

I formatted the wrong hard drive a few weeks back and lost every single phpBB file ever released from 3 weeks ago until the beginning of it's existence. I did data recovery but it is sketchy what I got in return. I will start digging in the files and see if that is one of them and if so if it is even a viable file anymore. I learned the hard way just because files are recovered does not always mean they are usable.

IntegraMOD commented 6 years ago

I uploaded the files I have on integramod.com

IntegraMOD commented 6 years ago

I have every file ever uploaded to integramod. The only problem is that depending on what version of pafiledb was running on what version of php, some are saved as the filename and some are saved as a hash

MWE001 commented 6 years ago

Nice! I was actually getting ready to jump on and have a look for the AGCM as well. I just walked in the door from a very long day of cardiologists and all that grown up crap. Fun times. I would rather pull out my hair trying to fix 15 year old code that was made for php 3 lmao!!

MWE001 commented 6 years ago

I found it:

##############################################################

MOD Title: Advanced Group Color Management for IntegraMOD 1.4.1

MOD Author: Phantomk < phantomk@modmybb.com > (n/a) http://www.modmybb.com/

IM Porter: Teelk < nerfherder20@hotmail.com > (Tim Beasley) http://www.e-tegra.com

MOD Description: Modify's the Group system to add colors.

MOD Version: 1.2.5

The file is in good shape. I even have the pre mod files as well.

vendethiel commented 6 years ago

I feel like such a mod so important (you even said it was the 1st mod people usually installed) should be included in IM. Do you agree?

MWE001 commented 6 years ago

Well, yes it probably should be. However, if I had to hazard a guess, I would say since the mod was not ported until after 1.4.1 was released that is the reason it was not included. Now I have had many beers and sleep since them days and I could very well be wrong but I am almost certain I am probably heading down correct avenue.

Helter uploaded all of his files and I am going to upload the one that was released as officially as well that contains the 1.4.1 pre-mod files as well. I will get a link to you here in a few minutes before I head out to yet another physicians visit.

vendethiel commented 6 years ago

I have both the versions, I'm comparing 309 and 379 ATM. From what I see we want the 379, maybe those are SVN revision numbers. One mentions Integra2 and the other fisubice. Other than that, 379 just seems to add color to more places.

MWE001 commented 6 years ago

@vendethiel I uploaded a file at Integramod as well.

vendethiel commented 6 years ago

@MWE001 Your file seems to be the same as 379.

I have implemented the mod! I also added it in places it wasn't at before, like the shoutbox: image

MWE001 commented 6 years ago

Nice! That's cool man. I like it.

vendethiel commented 6 years ago

I think I will replace its color picker however. I think it is pretty ugly and it could use a much better one, like you see nowadays.

MWE001 commented 6 years ago

Hey, is this very version you are working on the very version I can clone and open with my Git desktop or are you working on a local version? I ask because I have Integra installed and running perfectly fine on a Linux box and hate like hell to test and test and test only to find out I am 25 versions behind what you are building. lol Right off the bat my brain started going crazy finding things that needed fixed and my first thought was to see what version you are on. I did a sync from my desktop app with the master branch and it did not update. Now me being a newbie to Git still, is it syncing with my fork master branch or this very Integramod master Branch? I'm willing to say I made a rookie mistake and can answer my own question lol

MWE001 commented 6 years ago

Nope, I was right. It is this one I am fetching. It is showing no file changes at all.

Want me to hold up on my testing and messing about? In the mean time I will just install the old lottery mod and guest book and Garage mod maybe for the fun of it.

vendethiel commented 6 years ago

Hehe, I ‘m still gonna need to close off that other issue before I push. Also I have in this same repo my ADR branch.

MWE001 commented 6 years ago

Ok that's cool no worries. I was just not wanting to spend a bunch of time on something and it be a Pull or two behind. I suddenly have some time on my hands thanks to another health issue. Doctors orders to sit on my duff.

vendethiel commented 5 years ago

I'm fixing the last of it. Funnily enough my own system prepared groundwork for me to know where to fix things, even if I'm removing it now.

vendethiel commented 5 years ago

There are many uses of get_user_level_class that AGCM doesn't replace.. yet?

I feel like it should, but I need to see pages where the coloring is wrong to make sure.

MWE001 commented 5 years ago

I just got my site reinstalled and working properly (thats why you got my chmod.php you caught me right at a reinstall). I can run through every little place I know and test. I will create a account of every user class as well. The only one I probably couldn't do would be a donor maybe. I don't have a paypal sandbox setup.

MWE001 commented 5 years ago

I can leave my website virigin too and you can use it if you want to check pages. It isn't hurting anything sitting there. Just let me know and I can get in security and shift things around to allow for the accounts needed to test.

vendethiel commented 5 years ago

I'm going to commit my version with AGCM soon, it has a new (fairly hairy) SQL query to run. Not sure where I'm going to put it in the install procedure.

Here it is:

#
#--[ Alter Table ]-------------------------------------
#
ALTER TABLE phpbb_groups ADD group_weight MEDIUMINT(8) DEFAULT '0' NOT NULL;
ALTER TABLE phpbb_groups ADD group_legend SMALLINT(1) DEFAULT '0' NOT NULL;
ALTER TABLE phpbb_groups ADD group_color SMALLINT(1) DEFAULT '0' NOT NULL;

ALTER TABLE phpbb_users ADD user_group_id MEDIUMINT(8) DEFAULT 0 NOT NULL;

#
#--[ Colors Table ]-------------------------------------
#

CREATE TABLE phpbb_color (
  group_id mediumint(8) NOT NULL default '0',
  color_code varchar(6) NOT NULL default '',
  themes_id mediumint(8) NOT NULL default '0',
  hover_bold smallint(1) NOT NULL default '0',
  hover_italic smallint(1) NOT NULL default '0',
  hover_underline smallint(1) NOT NULL default '0',
  bold smallint(1) NOT NULL default '0',
  italic smallint(1) NOT NULL default '0',
  underline smallint(1) NOT NULL default '0',
  KEY group_id (group_id),
  KEY themes_id (themes_id)
);

#
#--[ Data ]---------------------------------------------
#
INSERT INTO phpbb_groups (group_type, group_name, group_description, group_moderator, group_single_user) VALUES (1, 'Group_registered', 'Group_registered_desc', 0, 1);
INSERT INTO phpbb_groups (group_type, group_name, group_description, group_moderator, group_single_user) VALUES (1, 'Group_anonymous', 'Group_anonymous_desc', 0, 1);
INSERT INTO phpbb_groups (group_type, group_name, group_description, group_moderator, group_single_user) VALUES (1, 'Group_session', 'Group_session_desc', 0, 1);

INSERT INTO phpbb_color (group_id, themes_id, color_code)
SELECT g.group_id, t.themes_id, ''
FROM phpbb_groups g, phpbb_themes t;

INSERT INTO phpbb_config (config_name , config_value) VALUES ('mod_advanced_group_color_management', '1.2.5');
INSERT INTO phpbb_config (config_name , config_value) VALUES ('group_registered', 'GROUP_REGISTERED');
INSERT INTO phpbb_config (config_name , config_value) VALUES ('group_anonymous', 'GROUP_ANONYMOUS');
INSERT INTO phpbb_config (config_name , config_value) VALUES ('group_session', 'GROUP_SESSION');
INSERT INTO phpbb_config (config_name , config_value) VALUES ('agcm_check', '0');
INSERT INTO phpbb_config (config_name , config_value) VALUES ('agcm_time', '2');
INSERT INTO phpbb_config (config_name , config_value) VALUES ('agcm_value', '1200');

UPDATE phpbb_config
  SET config_value = (SELECT group_id FROM phpbb_groups WHERE group_name = 'Group_registered')
  WHERE config_name = 'group_registered';
UPDATE phpbb_config
  SET config_value = (SELECT group_id FROM phpbb_groups WHERE group_name = 'Group_anonymous')
  WHERE config_name = 'group_anonymous';
UPDATE phpbb_config
  SET config_value = (SELECT group_id FROM phpbb_groups WHERE group_name = 'Group_session')
  WHERE config_name = 'group_session';

UPDATE phpbb_config SET config_value = (SELECT group_id FROM phpbb_groups WHERE group_name = 'Group_registered')
  WHERE config_name = 'user_group_id';
UPDATE phpbb_users SET user_group_id = (SELECT group_id FROM phpbb_groups WHERE group_name = 'Group_registered');
UPDATE phpbb_users SET user_group_id = (SELECT group_id FROM phpbb_groups WHERE group_name = 'Group_anonymous')
  WHERE user_id = -1;
vendethiel commented 5 years ago

To be clear: I have not yet tested installing the whole IM with AGCM.

But this sql I just posted installs it if you already have an IM board running.

vendethiel commented 5 years ago

Just pushed, see if you can find bugs or places where it doesn't work.

MWE001 commented 5 years ago

Ok. I'll have to see what I can do. I am fresh out of room on my live server and due to the database issues can not get it to run on my local host server. I'll give it a try one way or the other today.

MWE001 commented 5 years ago

Ok I'm uploading and installing this commit right as I type this.

It is on my production server. Same server specs as I posted a few days ago over at Integra website.

MWE001 commented 5 years ago

Uploaded all files, ran Chmod and double checked all permissions and ran the install. On initial run, it ran fine. It showed me all tables were installed in the database and as usual, asked me to remove the install directory before preceding to install Prill. So I did. Once I removed the install directory and clicked Install Prill, it redirected me to login.php and then give me a 500 internal server error and said my server couldn't handle that request.

So, I visited a few other websites on the server and all were delivering pages and login forms via SSL and unprotected just fine.

Tried a second install with this commit, same thing once again.

MWE001 commented 5 years ago

Tried leaving install file in and moving on knowing it was going to not work and it didn't work. It will not go past delete the install file. And there is no logging in at all. Message die error happens. Incomplete install.

vendethiel commented 5 years ago

Are you running PHP 7.2? or Mysql higher than 5.4?

MWE001 commented 5 years ago

Yeah that's the problem. mySQL 5.6 I have to stick with the other version of IM. That version of mySQL is to old. my WAMP stack won't go back that far nor will Site 5 allow me to roll back that far either.

Apache Version 2.2.31
PHP Version 5.6.35
MySQL Version 5.6.41-84.1
MWE001 commented 5 years ago

Yeah confirmed. That is all they will let me go. I can change php but sql will not allow me to change at all.

vendethiel commented 5 years ago

I have a patch where I have started to fix all SQL errors, have them work on more recent MySQL. Now that I've reinstalled MAMP with new versions of PHP & MySQL, I'll try to get a better version by next week. That probably means it won't work on ye olde versions but I'm fine with that.

MWE001 commented 5 years ago

I think unless someone has dedicated server they are being force fed newer stuff anyhow due to insecure scripts. So it might not be a big deal anyhow.

vendethiel commented 5 years ago

OK, I just installed it on my windows server. Only issue I had is that the install procedure took too long. I changed install's php time limit to 2minutes instead of 30 seconds and then it worked. Now I can confirm that AGCM works out of the box -- but I did the test on PHP5.5; the one I have installed here.

I'm going to close this issue. If someone finds an issue on PHP7+, please reopen and I'll fix it.

vendethiel commented 5 years ago

Seems like new users don't get automatically the "Registered User" color :(.

vendethiel commented 5 years ago

Sorry, PCP, I win this time :).

vendethiel commented 5 years ago

I was really annoyed by a "feature" of AGCM. You can define per-style colors, but that means if you pick "edit all styles", then it doesn't fill any default values (the current color, bold, etc, etc).

So I added a piece of code that queries all color info for all groups. And if all styles agree on a piece of data (i.e. all styles have group #1 = red), then it fills the default value with this value. Seems much simpler to use to me now. I'll push soon.