[RC] Wiki Coolness 1.0.1

[Pascal]

Moet het dan ook niet zo zijn met de wiki?

[FreakyBlue]

Zolang de user in config.php toegang tot phpBB heeft is dat geen probleem. Gewoon dit doen:

Code: Selecteer alles

SELECT * FROM db.table;

Is dat zo? Ook als de databases fysiek zijn gescheiden --> dus andere databasename, username en password

[Ger]

Is dat zo? Ook als de databases fysiek zijn gescheiden --> dus andere databasename, username en password

Username en password zullen vaak gelijk zijn (bij mij iig wel, heb het gewoon voor de overzichtlijkheid in 2 aparte databases staan en heb toch meer dan zat databases), maar vast lang niet altijd.

[Paul]

Zolang de user in config.php toegang tot phpBB heeft is dat geen probleem. Gewoon dit doen:

Code: Selecteer alles

SELECT * FROM db.table;

Is dat zo? Ook als de databases fysiek zijn gescheiden --> dus andere databasename, username en password

Alleen de username en password moeten hetzlfde zijn.

[FreakyBlue]

Vanavond aardig wat vooruitgang geboekt met de wikicredits.
Het database probleem is (bijna) opgelost. Verder zijn de manieren waarop je credits kan verdienen uitgebreid. Nu is het alleen mogelijk om op basis van 'harde edits' credits te verdienen. In de volgende release is dit uitgebreid met credits gebaseerd op:
- percentage edits
- harde toegevoegde text
- percentage toegevoegde text

Allemaal te managen via de ACP-modules die er bijgeleverd worden
Ik hoop [BETA]1.0.1 snel te kunnen releasen

[foxofinfinety]

Zolang de user in config.php toegang tot phpBB heeft is dat geen probleem. Gewoon dit doen:

Code: Selecteer alles

SELECT * FROM db.table;

Is dat zo? Ook als de databases fysiek zijn gescheiden --> dus andere databasename, username en password

Alleen de username en password moeten hetzlfde zijn.

dus beter:

Code: Selecteer alles

$phpbb_db = mysql_connect($phpbb_db_server, $phpbb_db_gebruikersnaam, $phpbb_db_wachtwoord);
$wiki_db = mysql_connect($wiki_db_server, $wiki_db_gebruikersnaam, $wiki_db_wachtwoord);
 

en dan via de ACP module de logins laten instellen
en dan die handles gebruiken waneer je voor elke iets moet doen dus:

Code: Selecteer alles

mysql_query(string Query, variabel Handle);
//voor een phpbb query:
mysql_query("SELECT * FROM phpbb_users", $phpbb_db);
//query op de wiki database:
mysql_query("SELECT * FROM pages", $wiki_db); 

[FreakyBlue]

Vers van de pers --> [BETA] wikicredits 2.0

Nieuw in deze release
- meerdere manieren om de wikicredits te verdienen (edits, tekst, combinatie | absoluut en percentage)
- automatische database installer en updater
- drie acp-modules waarin je de configuratie naar eigen behoefte kan aanpassen
- extra files om de MOD te installeren als je verschillende databases gebruikt

Kortom --> een geweldige update

Zie de eerste post van deze topic voor meerdere specificaties en screenshots. Mocht je de MOD gebruiken/waarderen --> dan wordt een kleine donatie voor alle uren die erin zitten ook gewaardeerd

Eventuele verzoeken voor future features --> geef ze maar door!!

[Paul]

Deze query kan onmogelijk correct zijn op deze manier;

Code: Selecteer alles

		// Let's do the second database thing needed for the edit percentage settings
		$sql = 'SELECT config_name, ss_total_edits
		FROM '.CONFIG_TABLE.', '.WIKISITE_STATS_TABLE.'';

Verder, besef je je dat je 4 * 25 queries extra toevoegt (met standaard phpBB settings) aan viewtopic.php? Je moet die queries joinen oid, niet in de while draaien.

Code: Selecteer alles

			set_config('wikicredits_based', request_var('wikicredits_based', '0'));
			set_config('wikicredits_show', request_var('wikicredits_show', '0'));

De tweede parameter van request_var is type specific.A ls het dus een getal moet zijn als uitkomst, moet de tweede parameter ook een getal zijn en geen string.

[FreakyBlue]

Deze query kan onmogelijk correct zijn op deze manier;

Code: Selecteer alles

		// Let's do the second database thing needed for the edit percentage settings
		$sql = 'SELECT config_name, ss_total_edits
		FROM '.CONFIG_TABLE.', '.WIKISITE_STATS_TABLE.'';

Uhm, dit is - volgens mij - helemaal correct Paul. Je roept de CONFIG_TABLE aan en de WIKISITE_STATS_TABLE en uit beide tabellen haal je info. Deze query kan je alleen draaien als beide (wiki en phpBB) in dezelfde database staan. Zo niet, dan gebruik je de extra files die ik heb toegevoegd (waar de queries zijn gescheiden).

Dus onderbouw aub je opmerking of zeg niets

Verder, besef je je dat je 4 * 25 queries extra toevoegt (met standaard phpBB settings) aan viewtopic.php? Je moet die queries joinen oid, niet in de while draaien.

Ik zie het probleem niet.

[Paul]

Deze query kan onmogelijk correct zijn op deze manier;

Code: Selecteer alles

		// Let's do the second database thing needed for the edit percentage settings
		$sql = 'SELECT config_name, ss_total_edits
		FROM '.CONFIG_TABLE.', '.WIKISITE_STATS_TABLE.'';

Uhm, dit is - volgens mij - helemaal correct Paul. Je roept de CONFIG_TABLE aan en de WIKISITE_STATS_TABLE en uit beide tabellen haal je info. Deze query kan je alleen draaien als beide (wiki en phpBB) in dezelfde database staan. Zo niet, dan gebruik je de extra files die ik heb toegevoegd (waar de queries zijn gescheiden).

Dus onderbouw aub je opmerking of zeg niets

Ik zou zeggen, probeer eens in phpmyadmin wat die query doet. Want zonder where gaat deze query echt niet iets doen wat jij wilt. Dat onmogelijk. Mischien nog een lesjes SQL nemen eerst als je het niet snapt .
En je kan wel doen alsof je beter weet als ik het weet, maar ik denk dat ik ook maar een klein beetje meer ervaring heb met het schrijven van MODs als jij hebt. Ook maar een heel klein beetje hoor.
En als je het dan nog niet snapt, draai dan voor de lol deze query eens, hetzelfde als die van jouw maar dan met een andere table:

Code: Selecteer alles

SELECT config_name, username FROM phpbb_config, phpbb_users;

En zorg ervoor dat je het niet op een board doet met teveel gebruikers. Ik zou het zielig vinden als je je hosts mysql server laat crashen.

Verder, besef je je dat je 4 * 25 queries extra toevoegt (met standaard phpBB settings) aan viewtopic.php? Je moet die queries joinen oid, niet in de while draaien.

Ik zie het probleem niet.

Ik wel. En een hoop hosts ook als je een meer als gemiddeld aantal pageviews hebt. 100 queries per pagina vind de server echt niet leuk.

[FreakyBlue]

Je zal vast en zeker meer ervaring hebben (is ook niet zo moeilijk, want dit is mijn eerste MOD). Alleen het zou fijn zijn als je dan ook je ervaring en kennis deelt. En niet zo maar iets roept. Geloof me, dan zullen mensen veel beter naar je luisteren

Deze query draai ik met de volgende reden:

Code: Selecteer alles

          // Let's do the second database thing needed for the edit percentage settings
          $sql = 'SELECT config_name, ss_total_edits
          FROM '.CONFIG_TABLE.', '.WIKISITE_STATS_TABLE.'';

ss_total_edits heb ik nodig om alles om te kunnen zetten naar percentages:

Code: Selecteer alles

$wikicredits_editspercentage = ($row['user_editcount'] / $row1['ss_total_edits'] * 100);

config_name selecteer ik om later deze code goed te laten werken:

Code: Selecteer alles

if (($config['wikicredits_based'] == 1) && ($config['wikicredits_show'] == 2)) {
		
			if (($wikicredits_editspercentage >= $config['wikiedits_k']) &&  ($wikicredits_editspercentage < $config['wikiedits_l'])) 
			{	
				$wiki_credits =  str_repeat($user->img('icon_wiki_credits', 'WIKI_CREDITS'), $row);
			}

Dan gaat het dus om

Code: Selecteer alles

config['wikicredits_based']

,

Code: Selecteer alles

$config['wikicredits_show']

en deze

Code: Selecteer alles

$config['wikiedits_k']

Wat klopt er dan niet?

[Paul]

alles, want je gebruikt heel $row['config_value'] niet. Heb je, zoals ik al zei, die query gedraaid die ik gegeven hebt. Doet precies hetzelfde als die van jouw alleen dan op de users table.
Mij is het al gelukt om een server met 3gig aan ram te laten crashen met die query .

[FreakyBlue]

Oke, ik denk dat ik je ga begrijpen

Tip is dus om de sql queries meer te verfijnen in de zin van

Code: Selecteer alles

SELECT config_name, config_value
          FROM phpbb_config
WHERE config_name = 'wikicredits_show'
or config_name = 'wikicredits_based'
etc...
;

[Paul]

Je gebruikt uberhaupt die waarde niet uit de config table, dus die kan je gewoon uit die query halen. Je gebruikt namelijk $config en niet $row (wat uit de query komt).

[FreakyBlue]

Je gebruikt uberhaupt die waarde niet uit de config table, dus die kan je gewoon uit die query halen. Je gebruikt namelijk $config en niet $row (wat uit de query komt).

Daar heb ik wat aan
Je hebt helemaal gelijk. Is me niet eens opgevallen ... die query kan er idd gewoon uit.
Neem ik mee in de next release.

Nog meer Paul

[Paul]

Je hebt een heleboel dubbele of haast dubbele code, ook niet echt wenselijk over het algemeen .

[FreakyBlue]

Je hebt een heleboel dubbele of haast dubbele code, ook niet echt wenselijk over het algemeen .

True. Maar volgens mij heb ik die "haast" dezelfde code nodig. Er zit steeds een klein verschil in, maar dat kleine verschil zorgt voor een compleet andere (wenselijke) uitkomst

In ieder geval thnx voor je opmerkingen/tips.

[FreakyBlue]

Nieuwe release: [BETA] 2.1

Updates:
- onnodige database queries eruit gehaald
- string issue <--> getal aangepast in acp_wikicredits.php

[FreakyBlue]

Deze week is de Release Candidate uitgegeven. Wijzigingen t.o.v. de eerdere versie 2.1:
- naam veranderd (incl. alle code) in wiki coolness
- files gestructureerd
- nieuwe images

Het nieuwe pakket is de downloaden op IDLaunch (zie eerste post van dit topic).

[Ger]

Vraagje, heb je deze ook al getest op 3.0.6.RCx?
Wil deze MOD in principe wel gaan gebruiken, maar dan moet ik er wel vanuit kunnen gaan dat het straks ook nog goed werkt. Op basis van de beschrijving verwacht ik geen problemen, maar beter safe then sorry.