Magento Varien Files – How to Patch or Rewrite Files in lib/Varien

adminformform-validationpatchesvarienvarien-file-uploader

I would like to know people's thoughts on the best way to patch/rewrite Varien lib files. Specifically whey looking into a couple of issues one with checkboxes and one with image type inputs I found that both these Varien elements have what I would deem as bugs.

So my What is the upgrade safe way of updating the following files?

Varien_Data_Form_Element_Image
Varien_Data_Form_Element_Checkboxes

Best Answer

Unfortunately there is no way to override these classes.
What you can do is treat them as abstract core classes.
Copy them to the local folder and make your changes there.
There is also the long and painful option of creating your own classes and changing the code that uses them to use your classes.
But that may end badly. It may not even be possible.

Related Solutions

Varien File Uploader – How to Upload Multiple Files

Magento uses Varien_File_Uploader to upload files within the system. When you want to upload multiple files, then use above class in a foreach loop. As far as I know, there is no "Magento way" for uploading multiple files. So go on with "PHP Way". ie use a foreach loop. This is an example

$_files = array(
    'userfile' => array(
        'allowed_extensions' => array('png', 'jpg', 'gif'),
        'rename_file'        => true,
        'file_dispersion'    => true,
        'file_path'          => 'path/to/the/file'
    ),
    'coverimage' => array(
        'allowed_extensions' => array('png', 'jpg', 'gif'),
        'rename_file'        => true,
        'file_dispersion'    => true,
        'file_path'          => 'path/to/the/file'
    )
);
foreach ($_files as $filename => $fileOpt) {
    if (isset($_FILES[$filename]['name']) && $_FILES[$filename]['name'] != '') {
        try {
            $uploader = new Varien_File_Uploader($filename);
            $uploader->setAllowedExtensions(array($fileOpt['allowed_extensions']));
            $uploader->setAllowRenameFiles($fileOpt['rename_file']);
            $uploader->setFilesDispersion($fileOpt['file_dispersion']);
            //$currentTimestamp = Mage::getModel('core/date')->timestamp(time()); 
            //$folder_name_date = date('Y_m_d', $currentTimestamp);
            //$path = Mage::getBaseDir('media') . DS . 'uploads' . DS . $folder_name_date;
            if(!is_dir($fileOpt['file_path'])){
                mkdir($path, 0777, true);
            }
            $uploader->save($fileOpt['file_path'], $_FILES[$filename]['name'] );             
            $newFilename = $uploader->getUploadedFileName();
            echo $fileOpt['file_path']. DS .$newFilename;
        } catch (Exception $e) {
            echo "Error";
        }
    } else {
        echo 'File is not set';
    }
}

Magento – Fixing extensions for SUPEE-6788

I would recommend using the https://github.com/rhoerr/supee-6788-toolbox tool from Paradox Labs. It scans your codebase for each of the issues listed in your question.

Best Answer

Related Solutions

Varien File Uploader – How to Upload Multiple Files

Magento – Fixing extensions for SUPEE-6788

Related Topic