search

williamleonard / obblm

Automatically exported from code.google.com/p/obblm
1 stars 0 forks source link

Cyanide: Option for team color #441

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
With the current leegmgr export for cyanide rosters the teams are always 
allocated 51 as their team color.

Is it possible to add an extra field in the teams database to include a value 
for the team color?

Then we could allow an input at team creation to specify the color and export 
it in the xml_botocs.php file.

Original issue reported on code.google.com by fellowpe...@gmail.com on 17 Jun 2010 at 12:41

GoogleCodeExporter commented 9 years ago 
This could also be handled without being in the DB as a choosable color upon 
download...

Original comment by funnyfin...@hotmail.com on 17 Jun 2010 at 1:58

GoogleCodeExporter commented 9 years ago 
You are right.

I think if the larger user base of OBBLM is not Cyanide Users then that makes 
sense.

May I ask, what are the legalities behind changing OBBLM to run as I want it on 
my own webspace? I have gotten it set up so that a team may choose a colour 
upon creation which is then stored in the team table and recalled upon 
exporting. I appreciate this might not be suitable for the BOTOCs guys to have 
to see with so can't be included in the main source but am I able to keep my 
site like that without breaching any laws? However this suits me for the 
purpose I am using the site which is to run a cyanide league.

The code I have used is ugly as I have never written any before but it works.

Site URL: http://labbl.unionenglish.net

Dummy user so people can check it out with id dummy and password dummy

Original comment by fellowpe...@gmail.com on 17 Jun 2010 at 6:38

GoogleCodeExporter commented 9 years ago 
The OBBLM guys have been very supportive in helping me setup my own websites 
for Cyanide-based leagues.  The only thing I'd suggest is to work really 
closely with these guys so your code gets integrated (and cleaned up perhaps) 
and that it allows you to have an easier time keeping your site updated as well.

As for the team Logo/Color options.. I'd love for them to be available.

--Micah

Original comment by micah.im...@gmail.com on 17 Jun 2010 at 3:47

GoogleCodeExporter commented 9 years ago 
I know it /can/ be done that way Funny but it's a tad strange and annoying to 
change your team colors and logos that way every time. Plus I wouldn't really 
want to tempt players by telling them to go into team edit all the time.

BASH is looking to start up again for the fall season. Being able to set colors 
and logos, in that order, is the #1 thing I'm looking for right now.

Original comment by furl...@gmail.com on 29 Jul 2010 at 2:53

GoogleCodeExporter commented 9 years ago

Original comment by Nimda...@gmail.com on 19 Oct 2010 at 10:03

GoogleCodeExporter commented 9 years ago 
What about randomizing the selected color? Would that be OK?

What is the color range? 1 to 51?

Original comment by Nimda...@gmail.com on 22 Oct 2010 at 10:54

GoogleCodeExporter commented 9 years ago 
Check out http://www.stuntyleeg.com for my solution to this.  If you like it I 
will put in all the colors and maybe a colorbox to click instead...

Original comment by funnyfin...@hotmail.com on 20 Nov 2010 at 8:54

GoogleCodeExporter commented 9 years ago 
BTW - colors are actually 0 through 55

Original comment by funnyfin...@hotmail.com on 20 Nov 2010 at 9:31

GoogleCodeExporter commented 9 years ago 
Comments?

Original comment by funnyfin...@hotmail.com on 24 Nov 2010 at 5:30

GoogleCodeExporter commented 9 years ago 
Where should I look?

Original comment by Nimda...@gmail.com on 27 Nov 2010 at 10:57

GoogleCodeExporter commented 9 years ago 
Sorry you have to choose a league with Cyanide enabled like PCBB and then look 
at the team page under rosters.

Original comment by funnyfin...@hotmail.com on 2 Dec 2010 at 12:48

GoogleCodeExporter commented 9 years ago 
Did I ever upload this?

Original comment by funnyfin...@hotmail.com on 22 Mar 2011 at 1:15

GoogleCodeExporter commented 9 years ago 
Don't know :-). Maybe not.

Original comment by Nimda...@gmail.com on 22 Mar 2011 at 3:28

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r773.

This is just the back end to the color code change.  Notice it adds a GET for 
cy and color code.

Original comment by funnyfin...@hotmail.com on 25 Mar 2011 at 11:49

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r774.

Ok this is the front end for it.  It adds a color chooser to choose the team 
color.  Let me know your thoughts.

Original comment by funnyfin...@hotmail.com on 25 Mar 2011 at 11:56

GoogleCodeExporter commented 9 years ago 
Any thoughts?

Original comment by funnyfin...@hotmail.com on 17 Apr 2011 at 2:04

GoogleCodeExporter commented 9 years ago 
I've edited a bit, but it looks like the cyanide roste rcreation is broke. 
Could you look at it soon William?

Original comment by Nimda...@gmail.com on 17 Apr 2011 at 10:46

GoogleCodeExporter commented 9 years ago 
r780 ...

Original comment by Nimda...@gmail.com on 17 Apr 2011 at 10:47

GoogleCodeExporter commented 9 years ago 
Did it work before your change?  It tested fine for me...  I'll take a look 
though.

Original comment by funnyfin...@hotmail.com on 17 Apr 2011 at 10:48

GoogleCodeExporter commented 9 years ago 
no it didn't actually.
Also, you may note that the actual link to the cyanide generating code is not 
changed (nor did I change your roster code).

Original comment by Nimda...@gmail.com on 17 Apr 2011 at 10:52

GoogleCodeExporter commented 9 years ago 
It needed that div in it to work I believe.  I had a hard time getting it to 
work precisley.  Why did you take away the color choice and only want to give 8 
colors instead of 56?

Original comment by funnyfin...@hotmail.com on 17 Apr 2011 at 10:52

GoogleCodeExporter commented 9 years ago 
You took out the colors and the entire style portion, can you email or IRC me 
your URL with these live changes?

Original comment by funnyfin...@hotmail.com on 17 Apr 2011 at 10:58

GoogleCodeExporter commented 9 years ago 
- The menu looks awful with that many entries.
- 8 colors should be enough, I mean the different shades don't matter.

Can we just try it with 8 to start with? I'm not veto'ing it or anything :-)

it should not be the DIVs fault that the cyanide roster code produces errors..?

Original comment by Nimda...@gmail.com on 17 Apr 2011 at 10:58

GoogleCodeExporter commented 9 years ago 
sorry, I have moved a few motnhs ago and my comp is behind a router with 
another webserver...

Original comment by Nimda...@gmail.com on 17 Apr 2011 at 11:00

GoogleCodeExporter commented 9 years ago 
Do you have a few to chat in IRC?  I just don't see how your roster doesn't 
work?  You could give me some full details here but may be easier to chat for a 
minute.

Original comment by funnyfin...@hotmail.com on 17 Apr 2011 at 11:05

GoogleCodeExporter commented 9 years ago 
Chat cleared this up.  You didn't have the PDO driver installed.  We can add a 
check for a function that should exist if the PDO is installed.

Original comment by funnyfin...@hotmail.com on 17 Apr 2011 at 11:21

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r782.

I didn't realize I forgot to upload the image.  Maybe this is why you didn't 
like what I submitted...  If you could revert back or just check out 
stuntyleeg, maybe you will like how I had it better.  I don't like how you have 
it as a list, the graphical presentation is nicer I think...

Original comment by funnyfin...@hotmail.com on 18 Apr 2011 at 1:03

GoogleCodeExporter commented 9 years ago 
Good solution,

r783

Original comment by Nimda...@gmail.com on 18 Apr 2011 at 11:22