Subsitution cleanup && test enhancement

This commit is contained in:
Nicolas Le Goff
2013-03-21 17:10:46 +01:00
committed by Romain Neutron
parent 93aa9be0fa
commit 02e5e2c216
4 changed files with 70 additions and 68 deletions

View File

@@ -127,30 +127,29 @@ class Tools implements ControllerProviderInterface
$controllers->post('/hddoc/', function(Application $app, Request $request) {
$success = false;
$errorMessage = "";
$fileName = null;
$message = _('An error occured');
if ($file = $request->files->get('newHD')) {
if ($file->isValid()) {
$fileName = $file->getClientOriginalName();
$size = $file->getClientSize();
$tempoFile = tempnam(sys_get_temp_dir(), 'substit');
unlink($tempoFile);
mkdir($tempoFile);
$tempoFile = $tempoFile . DIRECTORY_SEPARATOR . $fileName;
copy($file->getPathname(), $tempoFile);
if ($file->getClientSize() && $fileName && $file->isValid()) {
try {
$record = new \record_adapter(
$request->get('sbas_id')
, $request->get('record_id')
$tmpFile = sprintf(
'%stmp/%s',
$app['Core']->getRegistry()->get('GV_RootPath'),
$fileName
);
$media = $app['Core']['mediavorus']->guess(new \SplFileInfo($tempoFile));
if (false === rename($file->getPathname(), $tmpFile)) {
throw new \Exception();
}
$record = new \record_adapter(
$request->get('sbas_id'),
$request->get('record_id')
);
$media = $app['Core']['mediavorus']->guess(new \SplFileInfo($tmpFile));
$record->substitute_subdef('document', $media);
@@ -159,50 +158,44 @@ class Tools implements ControllerProviderInterface
}
$success = true;
$message = _('Record document has been successfully substitued');
unlink($tmpFile);
} catch (\Exception $e) {
$errorMessage = $e->getMessage();
}
unlink($tempoFile);
rmdir(dirname($tempoFile));
unlink($file->getPathname());
}
} else {
$errorMessage = _('file is not valid');
$message = _('file is not valid');
}
}
$template = 'prod/actions/Tools/iframeUpload.html.twig';
$var = array(
'success' => $success
, 'fileName' => $fileName
, 'errorMessage' => $errorMessage
);
return new Response($app['Core']->getTwig()->render($template, $var));
/**
*
*/
return new Response($app['Core']->getTwig()->render('prod/actions/Tools/iframeUpload.html.twig', array(
'success' => $success,
'message' => $message,
)));
});
$controllers->post('/chgthumb/', function(Application $app, Request $request) {
$success = false;
$errorMessage = "";
$message = _('An error occured');
if ($file = $request->files->get('newThumb')) {
$size = $file->getClientSize();
$fileName = $file->getClientOriginalName();
if ($size && $fileName && $file->isValid()) {
if ($file->getClientSize() && $fileName && $file->isValid()) {
try {
$rootPath = $app['Core']->getRegistry()->get('GV_RootPath');
$tmpFile = $rootPath . 'tmp/' . $fileName;
rename($file->getPathname(), $tmpFile);
$tmpFile = sprintf(
'%stmp/%s',
$app['Core']->getRegistry()->get('GV_RootPath'),
$fileName
);
if (false === rename($file->getPathname(), $tmpFile)) {
throw new \Exception();
}
$record = new \record_adapter(
$request->get('sbas_id')
, $request->get('record_id')
$request->get('sbas_id'),
$request->get('record_id')
);
$media = $app['Core']['mediavorus']->guess(new \SplFileInfo($tmpFile));
@@ -210,21 +203,19 @@ class Tools implements ControllerProviderInterface
$record->substitute_subdef('thumbnail', $media);
$success = true;
$message = _('Record thumbnail has been successfully substitued');
unlink($tmpFile);
} catch (\Exception $e) {
$errorMessage = _('An error occured');
}
} else {
$errorMessage = _('file is not valid');
$message = _('file is not valid');
}
$template = 'prod/actions/Tools/iframeUpload.html.twig';
$var = array(
'success' => $success
, 'fileName' => $fileName
, 'errorMessage' => $errorMessage
);
return new Response($app['Core']->getTwig()->render($template, $var));
return new Response($app['Core']->getTwig()->render('prod/actions/Tools/iframeUpload.html.twig', array(
'success' => $success,
'message' => $message,
)));
}
});

View File

@@ -1,7 +1,4 @@
<div class='content'>
{% if not success %}
{% trans 'an error occured' %}
{{errorMessage}}
{% endif %}
<div class='content {% if not success %}text-error{% endif %}'>
{{ message }}
</div>

View File

@@ -288,7 +288,6 @@
<fieldset style='border:1px solid #999;padding:20px;'>
<legend style='color:#EEE'>&nbsp;<b>{% trans "substitution HD" %}</b>&nbsp;</legend>
<div>
<input type="hidden" name="MAX_FILE_SIZE" value="20000000" />
<input id='new-hd-file' name="newHD" type="file"/>
<br />
<label for="CCFNALP" class="checkbox">
@@ -321,7 +320,6 @@
<fieldset style='border:1px solid #999;padding:20px;'>
<legend style='color:#EEE'>&nbsp;<b>{% trans "substitution SD" %}</b>&nbsp;</legend>
<div>
<input type="hidden" name="MAX_FILE_SIZE" value="20000000" />
<input id='new-sd-file' name="newThumb" type="file" />
<input type="hidden" name="sbas_id" value="{{record.get_sbas_id()}}" />
<input type="hidden" name="record_id" value="{{record.get_record_id()}}" />

View File

@@ -35,11 +35,30 @@ class ControllerToolsTest extends \PhraseanetWebTestCaseAuthenticatedAbstract
return $app;
}
public function testRouteChangeDoc()
{
$record = static::$records['record_1'];
$crawler = $this->client->request('POST', '/tools/hddoc/', array(
'sbas_id' => $record->get_sbas_id(),
'record_id' => $record->get_record_id(),
), array(
'newHD' => new UploadedFile(
$this->tmpFile, 'KIKOO.JPG', 'image/jpg', 2000
)
));
$response = $this->client->getResponse();
$message = trim($crawler->filterXPath('//div')->text());
$this->assertEquals(200, $response->getStatusCode());
$this->assertEquals(_('Record document has been successfully substitued'), $message);
}
public function testRouteChangeThumb()
{
$record = static::$records['record_1'];
$this->client->request('POST', '/tools/chgthumb/', array(
$crawler = $this->client->request('POST', '/tools/chgthumb/', array(
'sbas_id' => $record->get_sbas_id(),
'record_id' => $record->get_record_id(),
), array(
@@ -49,11 +68,8 @@ class ControllerToolsTest extends \PhraseanetWebTestCaseAuthenticatedAbstract
));
$response = $this->client->getResponse();
$content = $response->getContent();
$message = trim($crawler->filterXPath('//div')->text());
$this->assertEquals(200, $response->getStatusCode());
$this->assertFalse(strpos(_('An error occured'), $content));
$this->assertFalse(strpos(_('file is not valid'), $content));
$this->assertEquals(_('Record thumbnail has been successfully substitued'), $message);
}
}