View Single Post
Old 07-08-2009, 01:34 PM   #37
dangarion
All Star Reserve
 
dangarion's Avatar
 
Join Date: Feb 2003
Location: Orange, CA
Posts: 569
Ok just going to free flow some observations and some suggestions. Just to give you a background I used to run BOSI for the RWBL as well, so I have used tools like this in the past.

It would be nice if the tool would show the SQL import progress. Especially since the import of a large league (that has the full MLB history data) takes upwards of 10 minutes.

Also if there is an issue with a SQL import is there a way to revert and do it again?

I'm also having some type of issue with the financials since my SQL Log keeps showing this for all the teams in the league.
Code:
LOADING: team_last_financials.mysql.sql ... 237
   insert into `team_last_financials` VALUES (31, 41306087, 29565975, 16372701, 0, 0, 0, 10000000, 79733000, 7200000, 0, 2630961, 0, 0, 0, 0.0000, 0, 0, 91200000, 0, 0, ,1,1);
 -> You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '1,1)' at line 1
   insert into `team_last_financials` VALUES (32, 44920259, 34783500, 21813997, 0, 0, 0, 10000000, 95662500, 7930000, 0, 2861163, 0, 0, 0, 0.0000, 0, 0, 108200000, 0, 0, ,1,1);
 -> You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '1,1)' at line 1
   insert into `team_last_financials` VALUES (33, 41306087, 29565975, 18711658, 0, 0, 0, 10000000, 81197000, 8970000, 0, 2630961, 0, 0, 0, 0.0000, 0, 0, 94500000, 0, 0, ,1,1);
 -> You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '1,1)' at line 1
   insert into `team_last_financials` VALUES (34, 51794560, 45218550, 25992582, 0, 0, 0, 10000000, 123155308, 7280000, 0, 2999106, 0, 0, 0, 0.0000, 0, 0, 135500000, 0, 0, ,1,1);
 -> You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '1,1)' at line 1
   insert into `team_last_financials` VALUES (35, 44920259, 34783500, 22734952, 0, 0, 0, 10000000, 97257500, 8000000, 0, 2861163, 0, 0, 0, 0.0000, 0, 0, 109900000, 0, 0, ,1,1);
 -> You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '1,1)' at line 1
   insert into `team_last_financials` VALUES (36, 57363462, 45218550, 31280296, 0, 0, 0, 10000000, 136253500, 7590000, 0, 3321567, 0, 0, 0, 0.0000, 0, 0, 149200000, 0, 0, ,1,1);
I know there are team filters but it would also be nice to see all the data without a filter, and even positional filters in the areas that it's possible (any area that has the drop downs to only via each team, career metrics, development, ratings).

I know you did it for security reasons, but all the blank fields on the admin screen can be sort of confusing to people, especially to those that are used to seeing information in fields like that listed in other web scripts (phpBB, etc). Maybe include some text that they are blank for security reasons, still difficult to tell if the settings are correct though, not sure what you could do.

Talking about security, I've been forced to move our league exports to a separate UNIX user on the server, and unfortunately that makes it impossible for the script to see the folders that user has access to (the commish user is the web access account, and the export user only has access to it's own folders, unsure if there is a way to work around that with the export tracker or not, don't worry about it, just thought I would mention it.) I'm guessing there wouldn't be a way for me to resolve this unless if I did it on my own by like running a cron job or something, but even that would be a PITA.

Overall this tool is awesome! Really like what I'm seeing, and love that it was integrated in with the existing league reports.
__________________
DanGarion
GM - Los Angeles Dodgers The Peanuts and Cracker Jack Baseball League
PCJBL!
dangarion is offline   Reply With Quote