Congegni/sintonia
7
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

cc-3936: Fixed bug where setDbColMetadata to use airtime metadata key names.

Browse source
This commit is contained in:
Rudi Grinberg 2012-08-27 15:09:55 -04:00
parent 589477c8fe
commit 8d7f3817ec
1 changed files with 10 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

12
airtime_mvc/application/models/StoredFile.php
View file

@ -155,7 +155,7 @@ class Application_Model_StoredFile
if(!$owner) { // no owner detected, we try to assign one.
// if MDATA_OWNER_ID is not set then we default to the
// first admin user we find
if (!array_key_exists('MDATA_KEY_OWNER_ID', $p_md)) {
if (!array_key_exists('owner_id', $p_md)) {
//$admins = Application_Model_User::getUsers(array('A'));
$admins = Application_Model_User::getUsersOfType('A');
if (count($admins) > 0) { // found admin => pick first one
@ -165,7 +165,7 @@ class Application_Model_StoredFile
// get the user by id and set it like that
else {
$user = CcSubjsQuery::create()
->findPk($p_md['MDATA_OWNER_ID']);
->findPk($p_md['owner_id']);
if ($user) {
$owner = $user;
}
@ -178,6 +178,12 @@ class Application_Model_StoredFile
'".$p_md['MDATA_KEY_FILEPATH']."'");
}
}
# We don't want to process owner_id in bulk because we already
# processed it in the code above. This is done because owner_id
# needs special handling
if (array_key_exists('owner_id', $p_md)) {
unset($p_md['owner_id']);
}
foreach ($p_md as $dbColumn => $mdValue) {
// don't blank out name, defaults to original filename on first
// insertion to database.
@ -797,6 +803,8 @@ class Application_Model_StoredFile
$chunk = isset($_REQUEST["chunk"]) ? $_REQUEST["chunk"] : 0;
$chunks = isset($_REQUEST["chunks"]) ? $_REQUEST["chunks"] : 0;
$fileName = isset($_REQUEST["name"]) ? $_REQUEST["name"] : '';
# TODO : should not log __FILE__ itself. there is general logging for
# this
Logging::info(__FILE__.":uploadFile(): filename=$fileName to $p_targetDir");
// Clean the fileName for security reasons
//this needs fixing for songs not in ascii.