stian/gallery3
1
0
Fork 0
mirror of https://github.com/Pathduck/gallery3.git synced 2026-08-06 07:01:30 -04:00
Code Issues Actions Packages Projects Releases Wiki Activity

Add the ability to replace the source data file in Item_Model::save().

Browse Source 
Refactor the rotate code in Quick_Controller to replace the data file,
and then have gallery_event::item_updated_data_file() pick up after
the change is saved, rebuild the image and handle album covers.  This
is much more portable than before and it will allow any mechanism (eg:
REST) to replace the source image.
This commit is contained in:
Bharat Mediratta
2010-08-07 22:18:28 -07:00
parent 1abf43d3f1
commit dfb095a262
6 changed files with 137 additions and 60 deletions
Download Patch File Download Diff File Expand all files Collapse all files
modules/gallery/tests/Item_Model_Test.php
+31
View File
@@ -384,4 +384,35 @@ class Item_Model_Test extends Gallery_Unit_Test_Case {
$this->assert_same($photo->id, $album->album_cover_item_id);
}
public function replace_data_file_test() {
// Random photo is modules/gallery/tests/test.jpg which is 1024x768 and 6232 bytes.
$photo = test::random_photo();
$this->assert_equal(1024, $photo->width);
$this->assert_equal(768, $photo->height);
$this->assert_equal(6232, filesize($photo->file_path()));
// Random photo is gallery/images/imagemagick.jpg is 114x118 and 20337 bytes
$photo->set_data_file(MODPATH . "gallery/images/imagemagick.jpg");
$photo->save();
$this->assert_equal(114, $photo->width);
$this->assert_equal(118, $photo->height);
$this->assert_equal(20337, filesize($photo->file_path()));
}
public function replacement_data_file_must_be_same_mime_type_test() {
// Random photo is modules/gallery/tests/test.jpg
$photo = test::random_photo();
$photo->set_data_file(MODPATH . "gallery/images/graphicsmagick.png");
try {
$photo->save();
} catch (ORM_Validation_Exception $e) {
$this->assert_same(array("name" => "cant_change_mime_type"), $e->validation->errors());
return; // pass
}
$this->assert_true(false, "Shouldn't get here");
}
}