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.

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

Uploalimit für Quelltexte von Studenten #321

Open
Garados007 opened this issue May 11, 2018 · 2 comments
Open

Uploalimit für Quelltexte von Studenten #321

Garados007 opened this issue May 11, 2018 · 2 comments
Assignees
Milestone

Comments

@Garados007
Copy link
Member

Garados007 commented May 11, 2018

Uploadlimit eventuell erhöhen.
Limit zu finden unter: logic/LTutor.php unter Zeile 845 folgend.

// die Umwandlung der Datei wird nur durchgeführt, wenn sie mindestens den mimeType text/
// besitzt und nicht über 20kb groß ist.
if (isset($newFile)){
    if (strpos($newFile['mimeType'],'text/')!==false && $newFile['fileSize']<=20000){
        $newFileSend[] = $newFile;
    } else {
        if( $newFile['fileSize']>20000){
            // wenn die Datei zu groß ist, wollen wir einen Hinweis
            $newFileData = new File();
            $newFileData->setBody(Language::Get('main','submissionSizeError', self::$langTemplate, array('maxSize'=>20)), true);
            $newFileSend[] = $newFileData;
        }
    }
} else {
    // wenn keine konkrete Datei festgelegt wurde, dann muss eine leere existieren,
    // für die Rückgabe des POST /pdf
    $newFile = array();
    $fileInfo['filename'] = "Korrektur";
}           
@Garados007 Garados007 added this to the version 0.7 milestone May 11, 2018
@Garados007 Garados007 self-assigned this May 11, 2018
@tilluhlig
Copy link
Member

Man könnte es auch als Veranstaltungsparameter umsetzen, sodass es pro Veranstaltung in der Veranstaltungsverwaltung konfigurierbar ist... allerdings sollte man bedenken, dass zu große Dateien die Erstellung eines Korrekturarchivs komplett blockieren könnten, sodass beliebig große Werte problematisch werden können... trotzdem kann man natürlich einen vernünftigen Wert festlegen und die Einstellung mit einem Warnhinweise versehen

@tilluhlig
Copy link
Member

a

@tilluhlig tilluhlig added the bug label May 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants