Merge remote-tracking branch 'robbt/fix/show_files_meet_criteria' into dev/merge-testing

This commit is contained in:
Lucas Bickel 2018-01-09 21:51:30 +01:00
commit b7b581e6ec
5 changed files with 14 additions and 39 deletions

View file

@ -57,18 +57,21 @@ class PlaylistController extends Zend_Controller_Action
return $obj; return $obj;
} }
private function createUpdateResponse($obj) private function createUpdateResponse($obj, $formIsValid = false)
{ {
$formatter = new LengthFormatter($obj->getLength()); $formatter = new LengthFormatter($obj->getLength());
$this->view->length = $formatter->format(); $this->view->length = $formatter->format();
$this->view->obj = $obj; $this->view->obj = $obj;
$this->view->contents = $obj->getContents(); $this->view->contents = $obj->getContents();
if ($formIsValid) {
$this->view->poolCount = $obj->getListofFilesMeetCriteria()['count'];
}
$this->view->showPoolCount = true;
$this->view->html = $this->view->render('playlist/update.phtml'); $this->view->html = $this->view->render('playlist/update.phtml');
$this->view->name = $obj->getName(); $this->view->name = $obj->getName();
$this->view->description = $obj->getDescription(); $this->view->description = $obj->getDescription();
$this->view->modified = $obj->getLastModified("U"); $this->view->modified = $obj->getLastModified("U");
unset($this->view->obj); unset($this->view->obj);
} }
@ -99,7 +102,6 @@ class PlaylistController extends Zend_Controller_Action
$form = new Application_Form_SmartBlockCriteria(); $form = new Application_Form_SmartBlockCriteria();
$form->removeDecorator('DtDdWrapper'); $form->removeDecorator('DtDdWrapper');
$form->startForm($obj->getId(), $formIsValid); $form->startForm($obj->getId(), $formIsValid);
$this->view->form = $form; $this->view->form = $form;
$this->view->obj = $obj; $this->view->obj = $obj;
//$this->view->type = "sb"; //$this->view->type = "sb";
@ -555,8 +557,7 @@ class PlaylistController extends Zend_Controller_Action
if ($form->isValid($params)) { if ($form->isValid($params)) {
$this->setPlaylistNameDescAction(); $this->setPlaylistNameDescAction();
$bl->saveSmartBlockCriteria($params['data']); $bl->saveSmartBlockCriteria($params['data']);
$this->createUpdateResponse($bl, true);
$this->createUpdateResponse($bl);
$this->view->result = 0; $this->view->result = 0;
/* /*
$result['html'] = $this->createFullResponse($bl, true, true); $result['html'] = $this->createFullResponse($bl, true, true);
@ -599,7 +600,7 @@ class PlaylistController extends Zend_Controller_Action
if ($form->isValid($params)) { if ($form->isValid($params)) {
$result = $bl->generateSmartBlock($params['data']); $result = $bl->generateSmartBlock($params['data']);
$this->view->result = $result['result']; $this->view->result = $result['result'];
$this->createUpdateResponse($bl); $this->createUpdateResponse($bl, true);
#$this->_helper->json->sendJson(array("result"=>0, "html"=>$this->createFullResponse($bl, true, true))); #$this->_helper->json->sendJson(array("result"=>0, "html"=>$this->createFullResponse($bl, true, true)));
} else { } else {
$this->view->obj = $bl; $this->view->obj = $bl;
@ -624,7 +625,7 @@ class PlaylistController extends Zend_Controller_Action
$result = $bl->shuffleSmartBlock(); $result = $bl->shuffleSmartBlock();
$this->view->result = $result["result"]; $this->view->result = $result["result"];
$this->createUpdateResponse($bl); $this->createUpdateResponse($bl,true);
/* /*
if ($result['result'] == 0) { if ($result['result'] == 0) {
@ -652,7 +653,7 @@ class PlaylistController extends Zend_Controller_Action
$result = $pl->shuffle(); $result = $pl->shuffle();
$this->view->result = $result["result"]; $this->view->result = $result["result"];
$this->createUpdateResponse($pl); $this->createUpdateResponse($pl,true);
/* /*
if ($result['result'] == 0) { if ($result['result'] == 0) {
$this->_helper->json->sendJson(array( $this->_helper->json->sendJson(array(

View file

@ -366,8 +366,7 @@ class Application_Form_SmartBlockCriteria extends Zend_Form_SubForm
$this->setDecorators(array( $this->setDecorators(array(
array('ViewScript', array('viewScript' => 'form/smart-block-criteria.phtml', "openOption"=> $openSmartBlockOption, array('ViewScript', array('viewScript' => 'form/smart-block-criteria.phtml', "openOption"=> $openSmartBlockOption,
'criteriasLength' => count($this->getCriteriaOptions()), 'poolCount' => $files['count'], 'modRowMap' => $modRowMap, 'criteriasLength' => count($this->getCriteriaOptions()), 'modRowMap' => $modRowMap))
'showPoolCount' => $showPoolCount))
)); ));
} }

View file

@ -119,35 +119,6 @@
<?php } ?> <?php } ?>
</dd> </dd>
<?php if ($this->showPoolCount) { ?>
<div class='sp_text_font sp_text_font_bold'>
<span id='sp_pool_count' class='sp_text_font sp_text_font_bold'>
<?php
if ($this->poolCount > 1) {
echo $this->poolCount;
?>
<?php echo _("files meet the criteria")?>
</span>
<span class='checked-icon sp-checked-icon' id='sp_pool_count_icon'></span>
<?php
} else if ($this->poolCount == 1) {
echo $this->poolCount;
?>
<?php echo _("file meets the criteria")?>
</span>
<span class='checked-icon sp-checked-icon' id='sp_pool_count_icon'></span>
<?php
} else {
?>
0 <?php echo " "._("files meet the criteria")?>
</span>
<span class='sp-warning-icon' id='sp_pool_count_icon'></span>
<?php
}
?>
</div>
<?php } ?>
</dl> </dl>
</form> </form>

View file

@ -123,6 +123,7 @@ if (($i < count($items) -1) && ($items[$i+1]['type'] == 0)) {
} }
echo $this->partial('playlist/set-fade.phtml', $vars); echo $this->partial('playlist/set-fade.phtml', $vars);
?> ?>
</div> </div>
<?php endif; ?> <?php endif; ?>

View file

@ -739,6 +739,7 @@ input.input_text.sp_extra_input_text{
.sp_text_font_bold{ .sp_text_font_bold{
font-weight: bold; font-weight: bold;
color: white;
} }
.sp-ui-button-icon-only { .sp-ui-button-icon-only {
@ -782,6 +783,8 @@ input.input_text.sp_extra_input_text{
.sp-closed{ .sp-closed{
border-width: 0 0 0 !important; border-width: 0 0 0 !important;
} }
/***** SMART BLOCK SPECIFIC STYLES END *****/ /***** SMART BLOCK SPECIFIC STYLES END *****/
label { label {