Ignore:
Timestamp:
10/10/09 21:45:49 (10 years ago)
Author:
Kaydeth_parpg
Message:

Ticket #2: Patch by Kaydeth. Updated the rest of the classes in the scripts directory to correct some obvious discrepancies with our coding standard. Still need to go through the common and objects sub directories. comment[s:trac, t:2]

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/game/scripts/world.py

    r312 r313  
    1616#   along with PARPG.  If not, see <http://www.gnu.org/licenses/>. 
    1717 
    18 import fife, time 
     18import time 
     19import fife 
    1920import pychan 
    2021from sounds import SoundEngine 
     
    4748           @param engine: A fife.Engine instance 
    4849           @return: None""" 
    49         super(World, self).__init__(engine, regMouse = True, regKeys = True) 
     50        super(World, self).__init__(engine, regMouse=True, regKeys=True) 
    5051        # self.engine is a fife.Engine object, not an Engine object 
    5152        self.engine = engine 
    52         self.eventmanager = engine.getEventManager() 
     53        self.event_manager = engine.getEventManager() 
    5354        self.quitFunction = None 
    5455         
     
    5960 
    6061        # self.map is a Map object, set to none here 
    61         self.activeMap = None 
     62        self.active_map = None 
    6263        self.maps = {} 
    6364 
     
    9798        self.data.load(*args, **kwargs) 
    9899 
    99     def loadMap(self, mapname, filename): 
     100    def loadMap(self, map_name, filename): 
    100101        """Loads a map and stores it under the given name in the maps list. 
    101            @type mapname: text 
    102            @param mapname: The name of the map to load  
     102           @type map_name: text 
     103           @param map_name: The name of the map to load  
    103104           @type filename: text 
    104105           @param filename: File which contains the map to be loaded 
    105106           @return: None 
    106107        """ 
    107         if not mapname in self.maps: 
     108        if not map_name in self.maps: 
    108109            """Need to set active map before we load it because the map  
    109110            loader uses call backs that expect to find an active map.  
     
    111112            """ 
    112113            map = Map(self.engine, self.data) 
    113             self.maps[mapname] = map         
    114             self.setActiveMap(mapname) 
     114            self.maps[map_name] = map         
     115            self.setActiveMap(map_name) 
    115116            map.load(filename) 
    116117     
    117     def setActiveMap(self, mapname): 
     118    def setActiveMap(self, map_name): 
    118119        """Sets the active map that is to be rendered. 
    119            @type mapname: text 
    120            @param mapname: The name of the map to load  
     120           @type map_name: text 
     121           @param map_name: The name of the map to load  
    121122           @return: None 
    122123        """ 
    123         self.activeMap = self.maps[mapname] 
    124         self.activeMap.makeActive() 
     124        self.active_map = self.maps[map_name] 
     125        self.active_map.makeActive() 
    125126 
    126127    def displayObjectText(self, obj, text): 
     
    140141           @return: None""" 
    141142        key = evt.getKey() 
    142         keyval = key.getValue() 
    143  
    144         if(keyval == key.Q): 
     143        key_val = key.getValue() 
     144 
     145        if(key_val == key.Q): 
    145146            # we need to quit the game 
    146147            self.hud.quitGame() 
    147         if(keyval == key.T): 
    148             self.activeMap.toggle_renderer('GridRenderer') 
    149         if(keyval == key.F1): 
     148        if(key_val == key.T): 
     149            self.active_map.toggle_renderer('GridRenderer') 
     150        if(key_val == key.F1): 
    150151            # display the help screen and pause the game 
    151152            self.hud.displayHelp() 
    152         if(keyval == key.F5): 
     153        if(key_val == key.F5): 
    153154            # logic would say we use similar code to above and toggle 
    154155            # logic here does not work, my friend :-( 
    155156            self.cord_render.setEnabled(not self.cord_render.isEnabled()) 
    156         if(keyval == key.F7): 
     157        if(key_val == key.F7): 
    157158            # F7 saves a screenshot to fife/clients/parpg/screenshots 
    158             t = "screenshots/screen-%s-%s.png" % (date.today().strftime('%Y-%m-%d'), 
    159                                                   time.strftime('%H-%M-%S')) 
     159            t = "screenshots/screen-%s-%s.png" % \ 
     160                    (date.today().strftime('%Y-%m-%d'),\ 
     161                    time.strftime('%H-%M-%S')) 
    160162            print "PARPG: Saved:",t 
    161163            self.engine.getRenderBackend().captureScreen(t) 
    162         if(keyval == key.F10): 
     164        if(key_val == key.F10): 
    163165            # F10 shows/hides the console 
    164166            self.engine.getGuiManager().getConsole().toggleShowHide() 
    165         if(keyval == key.I): 
     167        if(key_val == key.I): 
    166168            # I opens and closes the inventory 
    167169            self.hud.toggleInventory() 
    168         if(keyval == key.A): 
     170        if(key_val == key.A): 
    169171            # A adds a test action to the action box 
    170             # The test actions will follow this format: Action 1, Action 2, etc. 
     172            # The test actions will follow this format: Action 1, 
     173            # Action 2, etc. 
    171174            self.hud.addAction("Action " + str(self.action_number)) 
    172175            self.action_number += 1 
    173         if(keyval == key.ESCAPE): 
     176        if(key_val == key.ESCAPE): 
    174177            # Escape brings up the main menu 
    175178            self.hud.displayMenu() 
    176179            # Hide the quit menu  
    177             self.hud.quitWindow.hide() 
    178         if(keyval == key.M): 
     180            self.hud.quit_window.hide() 
     181        if(key_val == key.M): 
    179182            self.sounds.toggleMusic() 
    180183 
     
    186189           @param evt: The event that fife caught 
    187190           @return: None""" 
    188         self.hud.hideContextMenu() # hide the context menu in case it is displayed 
     191        self.hud.hideContextMenu() 
    189192        scr_point = fife.ScreenPoint(evt.getX(), evt.getY()) 
    190193        if(evt.getButton() == fife.MouseEvent.LEFT): 
     
    192195        elif(evt.getButton() == fife.MouseEvent.RIGHT): 
    193196            # is there an object here? 
    194             instances = self.activeMap.cameras['main'].getMatchingInstances(scr_point, self.activeMap.agent_layer) 
     197            instances = self.active_map.cameras['main'].\ 
     198                            getMatchingInstances(scr_point, \ 
     199                                                 self.active_map.agent_layer) 
    195200            info = None 
    196201            for inst in instances: 
     
    201206                    break 
    202207                 
    203             # take the menu items returned by the engine or show a default menu if no items     
    204             data = info or [["Walk", "Walk here", self.onWalk, self.getCoords(scr_point)]] 
     208            # take the menu items returned by the engine or show a 
     209            # default menu if no items     
     210            data = info or \ 
     211                [["Walk", "Walk here", self.onWalk, self.getCoords(scr_point)]] 
    205212            # show the menu 
    206213            self.hud.showContextMenu(data, (scr_point.x, scr_point.y)) 
     
    221228        """ 
    222229        coord = fife.DoublePoint3D(float(position[0]), float(position[1]), 0) 
    223         location = fife.Location(self.activeMap.agent_layer) 
     230        location = fife.Location(self.active_map.agent_layer) 
    224231        location.setMapCoordinates(coord) 
    225232        self.data.game_state.PC.teleport(location) 
     
    231238           @return: None""" 
    232239        click = fife.ScreenPoint(evt.getX(), evt.getY()) 
    233         i=self.activeMap.cameras['main'].getMatchingInstances(click, self.activeMap.agent_layer) 
     240        i=self.active_map.cameras['main'].getMatchingInstances(click, \ 
     241                                                self.active_map.agent_layer) 
    234242        # no object returns an empty tuple 
    235243        if(i != ()): 
     
    238246                if(self.data.objectActive(obj.getId())): 
    239247                    # yes, so outline  
    240                     self.activeMap.outline_render.addOutlined(obj, 0, 137, 255, 2) 
     248                    self.active_map.outline_render.addOutlined(obj, 0, \ 
     249                                                               137, 255, 2) 
    241250                    # get the text 
    242251                    item = self.data.objectActive(obj.getId()) 
    243                     if(item): 
     252                    if(item is not None): 
    244253                        self.displayObjectText(obj, item.name) 
    245254        else: 
    246255            # erase the outline 
    247             self.activeMap.outline_render.removeAllOutlines() 
     256            self.active_map.outline_render.removeAllOutlines() 
    248257 
    249258    def getCoords(self, click): 
     
    253262           @rtype: fife.Location 
    254263           @return: The map co-ords""" 
    255         coord = self.activeMap.cameras["main"].toMapCoordinates(click, False) 
     264        coord = self.active_map.cameras["main"].toMapCoordinates(click, False) 
    256265        coord.z = 0 
    257         location = fife.Location(self.activeMap.agent_layer) 
     266        location = fife.Location(self.active_map.agent_layer) 
    258267        location.setMapCoordinates(coord) 
    259268        return location 
Note: See TracChangeset for help on using the changeset viewer.