Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Addressing #35 #42

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Addressing #35 #42

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
4032daa
Changed '+' string concatenations to str format()
OscarEReyes Oct 22, 2017
0b687e9
Changed " to '
OscarEReyes Oct 22, 2017
25665c6
Changed " to '
OscarEReyes Oct 22, 2017
4d05492
Changed str concatenation to format()
OscarEReyes Oct 22, 2017
d0600af
Fixed single quotes inside single quotes.
OscarEReyes Oct 23, 2017
ae2ec48
Deleted extra quotation mark
OscarEReyes Oct 23, 2017
0ef535d
Changed one concatenation to format()
OscarEReyes Oct 23, 2017
384a2d7
Fixed errors from missing parentheses
OscarEReyes Oct 23, 2017
752a925
Fixed missing parentheses
OscarEReyes Oct 23, 2017
eace8c6
Delete django-filemanager.iml
OscarEReyes Oct 23, 2017
0ce1706
Delete modules.xml
OscarEReyes Oct 23, 2017
a3c43ff
Fixed lines over 110 characters and missing parentheses
OscarEReyes Oct 23, 2017
d72ab9a
Merge branch 'master' of https://github.com/OscarEReyes/django-filema…
OscarEReyes Oct 23, 2017
418c649
Fixed line too long
OscarEReyes Oct 23, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
124 changes: 36 additions & 88 deletions filemanager/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -62,21 +62,21 @@ def __init__(
def rename_if_exists(self, folder, file):
if folder[-1] != os.sep:
folder = folder + os.sep
if os.path.exists(folder + file):
if os.path.exists("{}{}".format(folder, file)):
if file.find('.') == -1:
# no extension
for i in range(1000):
if not os.path.exists(folder + file + '.' + str(i)):
if not os.path.exists('{}{}.{}'.format(folder, file, str(i))):
break
return file + '.' + str(i)
return "{}.{}".format(file, str(i))
else:
extension = file[file.rfind('.'):]
name = file[:file.rfind('.')]
for i in range(1000):
full_path = folder + name + '.' + str(i) + extension
full_path = '{}{}.{}{}'.format(folder, name, str(i), extension)
if not os.path.exists(full_path):
break
return name + '.' + str(i) + extension
return '{}.{}{}'.format(name, str(i), extension)
else:
return file

Expand Down Expand Up @@ -106,7 +106,7 @@ def handle_form(self, form, files):
and not re.match(r'[\w\d_ -]+', name).group(0) == name
)
if invalid_folder_name:
messages.append("Invalid folder name : " + name)
messages.append('Invalid folder name : {}'.format(name))
return messages

invalid_file_name = (
Expand All @@ -118,12 +118,12 @@ def handle_form(self, form, files):
)
)
if invalid_file_name:
messages.append("Invalid file name : " + name)
messages.append('Invalid file name : {}'.format(name))
return messages

invalid_path = not re.match(r'[\w\d_ -/]+', path).group(0) == path
if invalid_path:
messages.append("Invalid path : " + path)
messages.append('Invalid path : {}'.format(path))
return messages
if action == 'upload':
for f in files.getlist('ufile'):
Expand All @@ -132,54 +132,33 @@ def handle_form(self, form, files):
or not re.match('[\w\d_ -/.]+', f.name).group(0) == f.name
)
if file_name_invalid:
messages.append("File name is not valid : " + f.name)
messages.append('File name is not valid : {}'.format(f.name))
elif f.size > self.maxfilesize*1024:
messages.append(
"File size exceeded "
+ str(self.maxfilesize)
+ " KB : "
+ f.name
)
messages.append('File size exceeded {} KB : {}'.format(str(self.maxfilesize), f.name))
elif (
settings.FILEMANAGER_CHECK_SPACE and
(
(self.get_size(self.basepath) + f.size)
> self.maxspace*1024
)
):
messages.append(
"Total Space size exceeded "
+ str(self.maxspace)
+ " KB : "
+ f.name
)
messages.append('Total Space size exceeded {} KB: {}'.format(str(self.maxspace), f.name))
elif (
self.extensions
and len(f.name.split('.')) > 1
and f.name.split('.')[-1] not in self.extensions
):
messages.append(
"File extension not allowed (."
+ f.name.split('.')[-1]
+ ") : "
+ f.name
)
name_split = f.name.split('.')[-1]
messages.append('File extension not allowed (.{}) : {}'.format(name_split, f.name))
elif (
self.extensions
and len(f.name.split('.')) == 1
and f.name.split('.')[-1]
not in self.extensions
):
messages.append(
"No file extension in uploaded file : "
+ f.name
)
messages.append('No file extension in uploaded file: {}'.format(f.name))
else:
filepath = (
self.basepath
+ path
+ self.rename_if_exists(self.basepath + path, f.name)
)
filepath = '{}{}{}'.format(self.basepath, path, f.name)
with open(filepath, 'w') as dest:
for chunk in f.chunks():
dest.write(chunk)
Expand All @@ -193,9 +172,9 @@ def handle_form(self, form, files):
try:
os.chdir(self.basepath + path)
os.mkdir(name)
messages.append('Folder created successfully : ' + name)
messages.append('Folder created successfully : {}'.format(name))
except:
messages.append('Folder couldn\'t be created : ' + name)
messages.append('Folder couldn\'t be created : '.format(name))
else:
messages.append(
'Folder couldn\' be created because maximum number of '
Expand All @@ -208,14 +187,9 @@ def handle_form(self, form, files):
try:
os.chdir(self.basepath + path)
os.rename(oldname, name)
messages.append(
'Folder renamed successfully from '
+ oldname
+ ' to '
+ name
)
messages.append('Folder renamed successfully from {} to {}'.format(oldname, name))
except:
messages.append('Folder couldn\'t renamed to ' + name)
messages.append('Folder couldn\'t renamed to {}'.format(name))
elif action == 'delete' and file_or_dir == 'dir':
if path == '/':
messages.append('root folder can\'t be deleted')
Expand All @@ -225,9 +199,9 @@ def handle_form(self, form, files):
try:
os.chdir(self.basepath + path)
shutil.rmtree(name)
messages.append('Folder deleted successfully : ' + name)
messages.append('Folder deleted successfully : {}'.format(name))
except:
messages.append('Folder couldn\'t deleted : ' + name)
messages.append('Folder couldn\'t deleted : {}'.format(name))
elif action == 'rename' and file_or_dir == 'file':
oldname = path.split('/')[-1]
old_ext = (
Expand All @@ -241,25 +215,14 @@ def handle_form(self, form, files):
try:
os.chdir(self.basepath + path)
os.rename(oldname, name)
messages.append(
'File renamed successfully from '
+ oldname
+ ' to '
+ name
)
messages.append('File renamed successfully from {} to {}'.format(oldname, name))
except:
messages.append('File couldn\'t be renamed to ' + name)
messages.append('File couldn\'t be renamed to {}'.format(name))
else:
if old_ext:
messages.append(
'File extension should be same : .'
+ old_ext
)
messages.append('File extension should be same : .{}'.format(old_ext))
else:
messages.append(
'New file extension didn\'t match with old file'
+ ' extension'
)
messages.append('New file extension didn\'t match with old file extension')
elif action == 'delete' and file_or_dir == 'file':
if path == '/':
messages.append('root folder can\'t be deleted')
Expand All @@ -269,25 +232,19 @@ def handle_form(self, form, files):
try:
os.chdir(self.basepath + path)
os.remove(name)
messages.append('File deleted successfully : ' + name)
messages.append('File deleted successfully : {}'.format(name))
except:
messages.append('File couldn\'t deleted : ' + name)
messages.append('File couldn\'t deleted : {}'.format(name))
elif action == 'move' or action == 'copy':
# from path to current_path
if self.current_path.find(path) == 0:
messages.append('Cannot move/copy to a child folder')
else:
path = os.path.normpath(path) # strip trailing slash if any
filename = (
self.basepath
+ self.current_path
+ os.path.basename(path)
)
filename = ('{}{}{}'.format(self.basepath, self.current_path, os.pathname(path)))
if os.path.exists(filename):
messages.append(
'ERROR: A file/folder with this name already exists in'
+ ' the destination folder.'
)
fe_error = 'A file/folder with this name already exists in the destination folder.'
messages.append('ERROR: {}'.format(fe_error))
else:
if action == 'move':
method = shutil.move
Expand All @@ -299,9 +256,7 @@ def handle_form(self, form, files):
try:
method(self.basepath + path, filename)
except:
messages.append(
'File/folder couldn\'t be moved/copied.'
)
messages.append('File/folder couldn\'t be moved/copied.')
return messages

def directory_structure(self):
Expand Down Expand Up @@ -356,25 +311,18 @@ def media(self, path):
w = width*60/mx
h = height*60/mx
img = img.resize((w, h), Image.ANTIALIAS)
response = HttpResponse(content_type=mimetype or "image/" + ext)
response = HttpResponse(content_type=mimetype or 'image/{}'.format(ext))
response['Cache-Control'] = 'max-age=3600'
img.save(
response,
mimetype.split('/')[1] if mimetype else ext.upper()
)
return response
except Exception:
imagepath = (
settings.FILEMANAGER_STATIC_ROOT
+ 'images/icons/'
+ ext
+ '.png'
)
imagepath = '{}images/icon/{}.png'.format(settings.FILEMANAGER_STATIC_ROOT, ext)

if not os.path.exists(imagepath):
imagepath = (
settings.FILEMANAGER_STATIC_ROOT
+ 'images/icons/default.png'
)
imagepath = '{}images/icons/default.png'.format(settings.FILEMANAGER_STATIC_ROOT)
img = Image.open(imagepath)
width, height = img.size
mx = max([width, height])
Expand All @@ -383,7 +331,7 @@ def media(self, path):
w = width*60/mx
h = height*60/mx
img = img.resize((w, h), Image.ANTIALIAS)
response = HttpResponse(content_type="image/png")
response = HttpResponse(content_type='image/png')
response['Cache-Control'] = 'max-age:3600'
img.save(response, 'png')
return response
Expand Down
2 changes: 1 addition & 1 deletion filemanager/settings.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
FILEMANAGER_STATIC_ROOT = getattr(
settings,
'FILEMANAGER_STATIC_ROOT',
os.path.dirname(os.path.abspath(__file__)) + '/static/filemanager/',
os.path.dirname('{}/static/filemanager/'.format(os.path.abspath(__file__))),
)
FILEMANAGER_CKEDITOR_JS = getattr(
settings,
Expand Down
4 changes: 2 additions & 2 deletions filemanager/widgets.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,9 +40,9 @@ def render(self, name, value, attrs=None):
div = "<div style='height:20px'></div>"
return rendered + mark_safe(
div
+ u"""<script type="text/javascript">
+ u'''<script type='text/javascript'>
document.addEventListener('DOMContentLoaded', function() {
CKEDITOR.replace('%s',%s);
}, false);
</script>""" % (attrs['id'], self.config)
</script>''' % (attrs['id'], self.config)
)