improve-file-dialog -- Icons for files

[deyan]

File dialog needs icons for at least supported:

• text files
• audio files
• video files
• s2 files
• pdf files
• html files
• zip files

It also needs icon for all other file types. You may research and eventually use LGPL artwork.

[deyan]

Batch update from file query.csv

[deyan]

Batch update from file active_tickets.csv

[tanya]

• The main.func.file module will provide extension point with FileIconProvider interface.
• FileIconProvider will have a method - getIcon():Icon
• Predefining FileView and setting it to the FileChooserDialog is what java provides for changing the icons.
• SophieFileView is the implementation of FileView for Sophie - the getIcon() method will be overriden to define what icons different files will have. What icons different file will have will be defined according to what their file extension is.
• For each file extension different class will be created that will implement FileIconPrivider.
• FileIcon will be the base class for all classes that will provide icons for different files. What is more, this class will have property for the icon that will be skinnable.

[pap]

• Singletons are not writen the way you have made FileView
• Make sure to make a branch for the implementation as you have binary resources added.
• It will be better if instead of file.getName().endsWith("ext") you use FileFilters to decide whether to use a specified icon. This way you can also make some implementation that will be easier to use.
  • This could be something like MyProvider.registerExtension(res, someFilter);
• The name "Icons" for a class, especially a top-level one is almost horrible. Also because of its lack of JavaDoc I cannot understand what its purpose is.
• The description of the design is not of very good quality. You should either write longer designs or have JavaDoc. Or best, have both.
• Please when you put a mark to the analysis explain briefly why did you select that exact mark.

[tanya]

• Singleton is fixed
• Branch is created - branches/private/tanya/fileicons 8800
• FileFilters are used
• Class Icons removed.
• JavaDoc added
• There are icons for:
  • Sophie Books
  • Sophie Resources
  • Default icon for files when non is available
  • Default icon for directories when non is available
  • Images
  • Videos
  • Audios
  • PDF documents
  • RTF documents
  • Plain text documents

[tanya]

8801
8809

[tanya]

• Icons added for:
  • Html files
  • Zip files

8816

[pap]

• Commited to trunk in [8838]
• You use loging bad in FileIconProviderPoint.getProvider method. You shouldn't be logging at WARN level in a loop like this. This should be a TRACE (as it is used to log the progress of a method) and you should set the minimum log level in FakeAuthorMain.
• The JavaDoc of the FileIcon class is bad. It say totally nothing.
• The name of the instance variable for the FileIconProvider extension point is bad - "dndDataExtensionPoint". :(
• I think that the FileIconProviderExtensionPoint.getProvider method would better be called findIcon and placed in SophieFileIcon but this doesn't seem very neccessary.
• Malformed JavaDoc in FileIcon - missing newlines.
• Very bad exception handling(almost swallowing) in FileIcon.register.
• Also the aforementioned method may be named better - "registerExtension"
• In the JavaDoc of FileIconProvider.getIcon you should mention that returning null means something.
• These FileFilters you used would better be placed in the corresponding FileIcon implementations instead. They won't be used in other places anyway. And also this way they can be made without any descriptions.
• I think that making FileIcon a singleton and then having lots of ancestors is not nice.
• FileIcon is designed so that its ancestors use the constructor with parameter so making it private is very very strange.
• Also if you have a constructor with parameters there is no need in hiding the default constructor like you did, as you don't have such unless it is explicitly defined.
• You should've added sort keys to extensions. Right now if I swap the places of the SophieBookIcon and SophieResourceIcon extensions in FileFunctionalityModule I get messed up icons.
• Also this way you can easily implement UnknownFileIcon and DirectoryIcon without that unnice singleton.
• You don't need to put public modifier in front of interface methods as it is put by default.
• Please have a look at the commited source code.

[meddle]

Closing all the tickets before M Y1