IRC log for #wowroster on 20080620

02:54.52*** join/#wowroster poetter (n=poetter@e177167242.adsl.alicedsl.de)
07:11.28*** join/#wowroster Tupsi (n=Miranda@217.7.101.202)
08:36.31*** join/#wowroster Chloe (n=vecna@clearstream.vecnix.net)
08:56.34*** join/#wowroster Aenix (n=vecna@clearstream.vecnix.net)
12:51.12*** join/#wowroster Nefuh (n=nefuh@dslb-084-060-226-164.pools.arcor-ip.net)
14:02.15*** join/#wowroster Zanix|Work (n=Zanix@71.216.154.51)
14:02.15*** mode/#wowroster [+o Zanix|Work] by ChanServ
14:02.41Zanix|Workhi Nefuh
14:02.48Nefuhhi Zanix
14:03.03Zanix|Workhows it going?
14:03.25Nefuhactual it seems that i have a solutionInformationslisteWert
14:03.25NefuhMotherboard NameAsus A7N8X Deluxe
14:03.33Nefuhsry wrong key pressed
14:03.53Zanix|Worklol
14:03.54Nefuhit seems that in my local roster the item will now be Full Parsed
14:04.02Zanix|WorkI used to have that exact board
14:04.11Zanix|Workno, I still do, in my server
14:04.14Zanix|Worklol
14:08.17NefuhZanix it seems that i found a solution for the item parser error
14:08.30Zanix|Workoh really?
14:08.32Zanix|Workthats cool
14:09.55Nefuhi´ve added a new preg_match that parse the mana regeneration string. Now the tooltip says Full Parsed and on the summary it will be displayed.
14:11.26Nefuhi´ll send you the changes i made so you can check it for yourself.
14:12.27Zanix|Workvery nice
14:19.20Nefuhso i send you a private message on wowroster.net. Maybe the Regex could be optimized i´m not so conform with regex.
14:23.18Zanix|Workk
15:01.27Zanix|Workhavent had a chance to look yet
15:01.32Zanix|Workbeen kinda busy hee
15:01.33Zanix|Workere
15:01.35Zanix|Workhere
15:02.22Nefuhok
15:18.34Zanix|WorkI looked at it
15:18.37Zanix|Workit looks sound
15:18.48Zanix|Workbut I didnt commit my last change to the svn last night
15:19.01Zanix|Workso I will have to package up Roster when I get home :(
15:42.29Zanix|Workok, I tweaked it a bit
15:45.05Zanix|Workwhat about health
15:45.11Zanix|Work$lang['tooltip_preg_manareg']='/Alle 5 Sek. (\d) (Mana|Health)/';
15:45.15Zanix|Workwould that be ok?
15:45.26Zanix|Workwait, no
15:49.52Zanix|Workthis is not going to pick up in other locales
15:50.55Zanix|Workonly dede is backwards
15:51.42Zanix|Workall other locales, the item parser in its current form would catch this
15:51.52Zanix|Worksince they all have + at the beginning
15:54.34Zanix|Workand this should be a base stat since we categorize "white" stats as base stats
16:01.39Nefuhbut for the basestats i need to change the string because the array for the basestats is like $tt['Attributes']['BaseStats']['Stamina'] = '+10 Stamina'
16:02.51Nefuhi could parse this tooltip string like $tt['Attributes']['BaseStats']['Mana'] = '+6 Mana per 5 seconds'
16:07.56*** join/#wowroster Nefuh_alt (n=jhufen@dslb-084-060-226-164.pools.arcor-ip.net)
16:20.19Zanix|Workhmmm
16:20.39Zanix|Workmaybe set a detection for a plus in the middle?
16:21.31Zanix|Workanother elseif below the first one
16:21.36Zanix|Workif( preg_match('/^([+-]\d+)\s(.+)/u', $line, $matches) )
16:21.46Zanix|Workuse this
16:21.46Zanix|Workelseif( preg_match('/^(.+)\s([+-]\d+)/u', $line, $matches) )
16:21.58Zanix|Workno, wait
16:22.00Zanix|Workdamn
16:22.15Zanix|Workthat wont catch and categprize it correctly
16:22.22Zanix|Workcategorize
16:22.23Zanix|Worklol
16:40.16Zanix|Workbasically,  do not want to make a special case for deDE
16:43.06*** join/#wowroster Chloe (n=vecna@clearstream.vecnix.net)
18:16.56Zanix|WorkNefuh: I think we need to just leave this out until 2.0.1 or something
18:17.24Nefuhok
18:17.30Zanix|Workwe REALLY need to release 2.0, it's been in development for a year now
18:17.39Zanix|Workso when I get home, I will package it up
18:17.48Zanix|Work16:30 is when I should get home
18:20.35Zanix|Workwill you be awake in 4 hours?
18:23.10Nefuhyeah i will stay here
18:23.34Zanix|Workk
18:23.48Zanix|Workif not, then you can provide a place for me to upload it or something
18:24.34Nefuhwait i will ask Luciver to assign a ftp account on our mirror server for you
18:25.31Zanix|WorkI might have access
18:25.37Zanix|Workvia a pm sent to me
18:25.44Zanix|Workdl.wowroster.de?
18:25.53Nefuhyes
18:28.11Zanix|WorkI have a pm with access to that
18:28.14Zanix|Workand it still works
18:28.21Zanix|Workill upload it there
18:28.27Nefuhok
18:31.53*** join/#wowroster Zanix|Work (n=Zanix@71.216.154.51)
18:31.53*** mode/#wowroster [+o Zanix|Work] by ChanServ
19:59.41*** join/#wowroster Chloe (n=vecna@clearstream.vecnix.net)
20:24.40NefuhHey zanix, why would the language selection on new install not set as default for roster? Is this a bug or normal?
20:46.45Zanix|Workit might be a bug
20:47.00Zanix|WorkNefuh: during the install where you enter the database information?
20:47.51Nefuhyes
20:48.25Zanix|Workyeah, it probably is a bug
20:48.48Nefuhand actual the class text coloring might not be functional for female chars, i currently look why but i can´t find a failure :(
20:50.08Nefuhthe female names should not be found in the array. And when i use my debug_array routine (looks nicer then print_r) on the $roster->locale->wordings[$language] sometimes there are only male class names
20:53.40Zanix|Workwhen in Roster, try using aprint()
20:53.45Zanix|Workit looks pretty too
20:54.06Zanix|Workoh, the female variations might be commented out
20:54.12Zanix|WorkI would check that
20:54.22Zanix|Workand there are something like 3 arrays for this....
20:59.23Zanix|Workgoing home early Nefuh, see ya later
21:00.05Nefuhyeah cu
21:05.08*** join/#wowroster Zanix-Work (n=Zanix@71.216.154.51)
21:08.56Zanix-Worksee ya
21:47.57Zanixand im home
21:49.36Nefuhnice
21:50.17Zanixoops
21:50.19Zanixlo
21:50.21Zanixlol
21:50.26Zanixgetting my channels mixed up
21:54.17Nefuhi repeat the installation of the roster and found that roster does an update of the config_value for the config_name locale. But this is set to enUS, but i´ve changed to german on the second step.
21:56.45Zanixit should be entering the chosen locale, after it parses the sql files
21:57.20Zanixah, the name= for the select box is missmatched
21:57.34Zanixin install.php, line 594
21:57.39Zanix$default_locale            = post_or_db('default_locale', $DEFAULTS);
21:57.46Zanixchange to
21:57.47Zanix$default_locale            = post_or_db('default_lang', $DEFAULTS);
21:58.23Nefuhah ok, seems that this was my fault. After adding the administrator password, the roster is in now in german.
22:00.09Nefuhthis was because on my first installation, i tried to use the install routine to create the database. But i have not defined a mysql password for the superuser and it seems that roster didn´t like this
22:02.42Zanixthe install.sh file?
22:03.00Nefuhno the web installer
22:04.01NefuhBut i see actual that roster set enUS as default language in the main configuration.
22:06.11Zanixyeah, the sql file has enUS in it
22:06.35Zanixbut it is supposed to take the locale selection from step 2 of the installer, but it was looking at the wrong post value
22:06.42Zanixit will be fixed
22:07.44Nefuhok
22:08.27Nefuhwhat is with the class coloring? I found a workaround, but i don´t now if it´s good.
22:11.55Nefuhyou use the php array_search function in /addons/memberlists/inc/memberslist.php near line 708 this seems not working. I replace it with $class_color = $roster->locale->wordings[$language]['class_to_en'][$row['class']]; this works in my local roster in german, but i don´t know what about other locales.
22:14.09Zanixyeah, that would be right
22:14.23Zanixjust need to check if the locale string exists before you try to use it
22:14.29Zanixor you get a notice error
22:18.40ZanixI made the change, and using your data, it looks like it works
22:18.49Zanixhttp://svnroster.clarkjo.com/roster/trunk/guild/memberslist/a=g:2.html
22:19.59Nefuhyes looks good
22:20.52Zanix$class_color = ( isset($roster->locale->wordings[$language]['class_to_en'][$row['class']]) ? $roster->locale->wordings[$language]['class_to_en'][$row['class']] : '' );
22:20.54Zanixthis is the line
22:21.20Nefuhk this looks complicated ;)
22:21.36Zanixlol
22:21.49Zanixbasically it is an if statement on one line
22:22.57Nefuhyeah i know but i found this is complicate to read, when you will debug this. a normal if statement is longer but i found it better to read, just my opinion.
22:26.52Zanixyeah
22:36.33Zanixits actually not that hard of a line
22:36.47Zanix( if true ? then do this : else do this )
22:38.01Nefuhah thanks for the info
22:41.56Zanixnp
22:42.18Zanixhttp://www.ilovejackdaniels.com/php/ternary-conditionals/
22:42.21Zanixfor more info

Generated by irclog2html.pl Modified by Tim Riker to work with infobot.