SolidOS / SolidOS/solid-ui

on resource creation : double check if contentType return same extension as file extension

Offen
#146 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
TypeScript
Sterne
153
Forks
46
Ø Merge
1 T. 8 Std.
Gemergte PRs (30 T.)
7

Beschreibung

see https://github.com/solid/folder-pane/issues/5

on resource creation solid-ui in https://github.com/solid/solid-ui/blob/master/src/widgets/dragAndDrop.js/uploadFiles check if contentType match file extension L170-173

     var extension = mime.extension(theFile.type)
     if (theFile.type !== mime.lookup(theFile.name)) {
       suffix = '_.' + extension
       console.log('MIME TYPE MISMATCH -- adding extension: ' + suffix)

The same control is made in NSSv5.2.0 in resource-mapper.mapUrlToFile L108-110
The control should be deleted from solid-ui

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne in solid-ui's src/widgets/dragAndDrop.js bei uploadFiles um die Zeilen 170–173 und vergleiche die MIME-type-Prüfung mit NSSv5.2.0 resource-mapper.mapUrlToFile bei den Zeilen 108–110. Bestätige, wie die Ressourcenerstellung derzeit mit der Erweiterung umgeht, und entferne dann die doppelte Kontrolle aus solid-ui. Erledigt ist dies, wenn die redundante Prüfung und ihr Suffix-Verhalten nicht mehr vorhanden sind.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
typescript
Bereich
frontend
Issue-Typ
Refactoring
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
42/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.