bgame-hunter / temars-eve-api

Automatically exported from code.google.com/p/temars-eve-api
0 stars 0 forks source link

an empty API field breaks when you have no characters and enter a new API. #37

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
TEA Version?
1.2.0

SMF Version?
2.0 RC5

What steps will reproduce the problem?
1. Enter an API under profile settings when no API info exists

What is the expected output? What do you see instead?
Expect to see a return to the page with characters selected, instead page dies 
on a white screen stating that character id must be numeric if selected

Please provide any additional information below.

I fixed this as follows.  

In TEA.php

                if(!empty($charlist))
                {
                        foreach($charlist as $i => $c)
                        {
                                echo '<option value="'.$i.'"',$i == $teainfo[0]['main'] ? 'SELECTED' : '','>'.$c.'</option>';
                        }
                }
                else
                {
                        echo '<option value="-", SELECTED>-</option>';
                }

The else clause was added after the if clause at line 2752.

Original issue reported on code.google.com by monorail...@gmail.com on 26 Apr 2011 at 12:24

GoogleCodeExporter commented 8 years ago

Original comment by tema...@googlemail.com on 27 Apr 2011 at 11:24

GoogleCodeExporter commented 8 years ago

Original comment by tema...@googlemail.com on 28 Apr 2011 at 3:05

GoogleCodeExporter commented 8 years ago
Issue 40 has been merged into this issue.

Original comment by tema...@googlemail.com on 28 Apr 2011 at 3:06

GoogleCodeExporter commented 8 years ago

Original comment by tema...@googlemail.com on 14 Aug 2011 at 3:07