upravy_exportu #91

Open
ticvac wants to merge 5 commits from upravy_exportu into master
2 changed files with 22 additions and 7 deletions
Showing only changes of commit c1df250a46 - Show all commits

View file

@ -6,6 +6,9 @@
<h2><strong>Export lidí</strong></h2>
<p>Vyberte pole, které chcete exportovat</p>
Review

Takže pokud přibyde na řešitelovi další věc, tak se bude muset přidávat sem další položka?

Takže pokud přibyde na řešitelovi další věc, tak se bude muset přidávat sem další položka?
Review

Ano

Ano
<!-- for loop zde neni pouzit proto, aby se mohlo napsat
data-value="email telefon mesto"
a zabalit tak vice parametru do jednoho checkboxu -->
<p>
<label>( Jméno: <input class="field-check" data-value="jmeno" type="checkbox" checked>)</label>
<label>( Příjmení: <input class="field-check" data-value="prijmeni" type="checkbox" checked>)</label>
@ -89,14 +92,20 @@
download_button.addEventListener('click', (e) => {
// uzivatele vybrana pole
const fields = Array.from(document.getElementsByClassName('field-check'))
fields = Array.from(document.getElementsByClassName('field-check'))
.filter(e => e.checked)
.map(e => e.getAttribute('data-value'))
console.log(fields)
.map(e => e.getAttribute('data-value'));
params = ""
for (let val of fields) {
for(let s of val.split(' ')) {
params += s + ","
}
}
params = params.slice(0, -1)
Review

Nechceš použít fields.join(",")? Tohle mě neuráží, ale .join zní jakože to ušetří 5 řádků a dvě logické chyby…

Nechceš použít `fields.join(",")`? Tohle mě neuráží, ale `.join` zní jakože to ušetří 5 řádků a dvě logické chyby…
if (select_two.innerHTML == '') {
window.location.href = "/profil/exporty_lidi/get_csv_only_one_step/" + select_one.value + "?fields=" + fields.join(',')
window.location.href = "/profil/exporty_lidi/get_csv_only_one_step/" + select_one.value + "?fields=" + params
} else {
window.location.href = "/profil/exporty_lidi/get_csv/" + select_one.value + "/" + select_two.value + "?fields=" + fields.join(',')
window.location.href = "/profil/exporty_lidi/get_csv/" + select_one.value + "/" + select_two.value + "?fields=" + params
}
})
</script>

View file

@ -171,8 +171,14 @@ def get_export_options(request, type):
data = [{"id": s.id, "display": str(s)} for s in Soustredeni.objects.all()]
return HttpResponse(json.dumps(data), content_type='application/json')
def download_export_csv_only_first_step(request, type):
def getFieldsForExport(request):
Review

Takže když nastala chyba, tak to udělá něco náhodného?

Takže když nastala chyba, tak to udělá něco náhodného?
Review

Jako... ona nenastala chyba, uživatel si zažádal o prázdné info... dostane to, co je nastaveno defaultně

Jako... ona nenastala chyba, uživatel si zažádal o prázdné info... dostane to, co je nastaveno defaultně
Review

To právě není úplně pravda. Někde nastala chyba, takže jsi dostal špatná data, a vracíš něco, co tomu neodpovídá.

To právě není úplně pravda. Někde nastala chyba, takže jsi dostal špatná data, a vracíš něco, co tomu neodpovídá.
fields = request.GET.get('fields').split(',')
if fields == ['']:
return ["jmeno", "prijmeni", "email", "telefon"]
return fields
def download_export_csv_only_first_step(request, type):
fields = getFieldsForExport(request)
if type == 3:
Review

Uhh, a když type != 3, tak se stane co? A co je vůbec _only_first_step sémanticky?

Uhh, a když `type != 3`, tak se stane co? A co je vůbec `_only_first_step` sémanticky?
resitele = tvorba_utils.resitele_co_neodmaturovali()
resiteleOsoby = Osoba.objects.filter(resitel__in=resitele)
@ -181,7 +187,7 @@ def download_export_csv_only_first_step(request, type):
return response
def download_export_csv(request, type, id):
fields = request.GET.get('fields').split(',')
fields = getFieldsForExport(request)
if type == PrvniTypExportu.CISLA.value:
resitele = tvorba_utils.resi_cislo(Cislo.objects.get(id=id))
resiteleOsoby = Osoba.objects.filter(resitel__in=resitele)