Ticket #217 (closed patch: fixed)

Opened 13 months ago

Last modified 13 months ago

Verify first that map exist in custom emoticon before opening it

Reported by: patrick.dessalle Owned by: somebody
Priority: minor Milestone: 1.0
Component: other Version:
Keywords: custom emoticons Cc:

Description

Hello,

The map file in the custom emoticon does not exist by default. Instead of trying to open it and throwing an exception if it does not exist, I suggest to try if the file exist first (using os.path.exists) and then trying to open it.

I've also slightly changed the code to generate the path to the map file only once in the function and to create the empty dict self.list outside of the try/except and not twice (once in the try, once in the except).

Attachments

checkMapExists.diff (1.7 kB) - added by patrick.dessalle 13 months ago.
modification to check that the map file exists before opening it.

Change History

Changed 13 months ago by patrick.dessalle

modification to check that the map file exists before opening it.

Changed 13 months ago by dx

  • status changed from new to closed
  • resolution set to fixed

applied

Add/Change #217 (Verify first that map exist in custom emoticon before opening it)

Author



Action
as closed
Next status will be 'reopened'
 
Note: See TracTickets for help on using tickets.