Skip to content
This repository was archived by the owner on Sep 10, 2021. It is now read-only.

Commit 61973a4

Browse files
author
Charles Ma
committed
BUG: fixed bug #200 Was a javascript error
Also fixed edit/create form Added some html5 elements
1 parent a535a3c commit 61973a4

File tree

6 files changed

+38
-9
lines changed

6 files changed

+38
-9
lines changed

core/controllers/FolderController.php

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,8 +55,8 @@ function editAction()
5555
{
5656
$name = $this->_getParam('name');
5757

58-
// Check ifa folder with the same name already exists for the same parent
59-
if($this->Folder->getFolderExists($name, $folder->getParent()))
58+
// Check if folder with the same name already exists for the same parent
59+
if($folder->getName() != $name && $this->Folder->getFolderExists($name, $folder->getParent()))
6060
{
6161
throw new Zend_Exception('This name is already used');
6262
}
@@ -239,6 +239,9 @@ public function createfolderAction()
239239
$folder_id = $this->_getParam('folderId');
240240
$folder = $this->Folder->load($folder_id);
241241
$header = "";
242+
$form = $this->Form->Folder->createEditForm();
243+
$formArray = $this->getFormAsArray($form);
244+
$this->view->form = $formArray;
242245
if(!isset($folder_id))
243246
{
244247
throw new Zend_Exception("Please set the folderId.");
@@ -265,6 +268,12 @@ public function createfolderAction()
265268
}
266269
else
267270
{
271+
// Check if folder with the same name already exists for the same parent
272+
if($this->Folder->getFolderExists($name, $folder))
273+
{
274+
echo JsonComponent::encode(array(false, $this->t('This name is already used')));
275+
return;
276+
}
268277
$new_folder = $this->Folder->createFolder($name, '', $folder);
269278
$policyGroup = $folder->getFolderpolicygroup();
270279
$policyUser = $folder->getFolderpolicyuser();

core/controllers/forms/FolderForm.php

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,11 +22,14 @@ public function createEditForm()
2222
->setMethod('post');
2323

2424
$name = new Zend_Form_Element_Text('name');
25-
$name ->setRequired(true)
25+
$name ->setAttribs(array('placeholder' => $this->t('Name of the folder'), 'autofocus' => 'autofocus', 'required' => 'required'))
26+
->setRequired(true)
2627
->addValidator('NotEmpty', true);
2728

2829
$description = new Zend_Form_Element_Textarea('description');
30+
$description ->setAttribs(array('placeholder' => $this->t('Optional')));
2931
$teaser = new Zend_Form_Element_Text('teaser');
32+
$teaser ->setAttribs(array('placeholder' => $this->t('Optional')));
3033
$teaser->setAttrib('MAXLENGTH', '250');
3134
$submit = new Zend_Form_Element_Submit('submit');
3235
$submit ->setLabel($this->t("Save"));
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
form#editFolderForm input[type="text"], form#editFolderForm input[type="password"] ,form#editFolderForm textarea
2+
{
3+
width: 350px!important;
4+
}
5+

core/public/js/common/common.browser.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -110,15 +110,15 @@
110110
if(jsonResponse[0])
111111
{
112112
createNotive(jsonResponse[1],1500);
113+
node.remove();
114+
$( "div.MainDialog" ).dialog('close');
113115
node.each(function(){
114116
var children = childrenOf($(this));
115-
if(children!=undefined)
117+
if(children != undefined)
116118
{
117119
children.remove();
118120
}
119-
});
120-
node.remove();
121-
$( "div.MainDialog" ).dialog('close');
121+
});
122122
}
123123
else
124124
{

core/translation/fr-main.csv

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -251,4 +251,8 @@ Add in group;Ajouter au groupe
251251
Set as moderator;Promouvoir en tant que mod�rateur
252252
Remove from community;Retirer de la communaut�
253253
Remove from group;Retirer du groupe
254-
Add Metadata;Ajouter M�tadonn�e
254+
Add Metadata;Ajouter M�tadonn�e
255+
This name is already used;Ce nom est d�ja utilis�
256+
Name of the folder;Nom du r�pertoire
257+
Optional;Optionel
258+
Teaser;Courte description

core/views/folder/createfolder.phtml

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,15 @@ echo '<script type="text/javascript" src="' . $this->coreWebroot . '/public/js/j
2020
<input type="hidden" name="folderId" value="<?php echo $this->parentFolder->getKey()?>"/>
2121
<div >
2222
<label for="name"><?php echo $this->t("Name")?></label>
23-
<input type='text' name="name"/>
23+
<?php echo $this->form['name']?>
24+
</div>
25+
<div >
26+
<label for="description"><?php echo $this->t("Description")?></label>
27+
<?php echo $this->form['description']?>
28+
</div>
29+
<div >
30+
<label for="teaser"><?php echo $this->t("Teaser")?></label>
31+
<?php echo $this->form['teaser']?>
2432
</div>
2533

2634
<div>

0 commit comments

Comments
 (0)