#311
  • Created on
    2011-08-19 10:36
  • Reported by
    phoenyx
  • Status
    Closed
  • Priority
    Normal (3)
  • Assigned to
    phoenyx
  • Milestone
    -
  • Plan Level
    -
  • Permission type
    None
  • Due Date
    -

Attachments

Related Tickets

Add new subtask or bug

Uploading Attachments

Currently it's not possible to upload attachments due to a misunderstanding.
In the forum settings you have to specify extensions whereas the uploader requires mimetypes.

The problem is a modification to the icms media uploader which was done for 1.3. All class functions are called with self:: which makes it impossible to overwrite a method in an extending class (such as iforum_uploader for iForum). Usually iforum has its own function to check the mimetype.

Finally, this is rather a core issue.
User picture
  on 2011-08-19 15:04 *
By skenow
Yes, if you extend a class and then use a method from the parent class that references self::, that refers to the parent class and not the originating class. In this case, you can override the parent method with your own method - add iforum_uploader::upload().
User picture
  on 2011-08-20 04:30 *
By phoenyx
Status changed from New to assigned
Good hint, thank you. However, too many things that I would need are not even protected so I would have to copy almost the whole uploader class. Will go for another solution.
User picture
  on 2011-08-20 04:30 *
By phoenyx
Status changed from assigned to closed
In revision:22274: Fixing* 787 by converting extensions to mimetypes via the core mimetype handler (requires the mimetype to be known however)
User picture
  on 2011-09-27 14:01 *
By phoenyx
Component changed from None to Addons - iForum
Updating tickets (#649, #771, #787, #788, #789, #790, #791, #792, #813, #814, #815)
User picture
  on 2011-09-29 19:04 *
By skenow
Component changed from Modules: Content to Modules: iForum
Updating tickets (#23, #298, #28, #278, #315, #317, #325, #326, #27, #186, #220, #254, #311, #312, #313, #314, #316, #319, #327, #24, #25, #26, #29, #30, #188, #189, #230, #236)