Merge branch '274-logging' into 'develop'

Resolve "Broken logging formatting in import task"

Closes #274 and #278

See merge request funkwhale/funkwhale!233
merge-requests/237/head
Eliot Berriot 2018-06-05 18:55:17 +00:00
commit 33ae51fc91
4 zmienionych plików z 10 dodań i 11 usunięć

Wyświetl plik

@ -259,7 +259,9 @@ def get_cover_from_fs(dir_path):
'import_job') 'import_job')
def import_job_run(self, import_job, replace=False, use_acoustid=False): def import_job_run(self, import_job, replace=False, use_acoustid=False):
def mark_errored(exc): def mark_errored(exc):
logger.error('[Import Job %s] Error during import: %s', str(exc)) logger.error(
'[Import Job %s] Error during import: %s',
import_job.pk, str(exc))
import_job.status = 'errored' import_job.status = 'errored'
import_job.save(update_fields=['status']) import_job.save(update_fields=['status'])

Wyświetl plik

@ -0,0 +1 @@
Broken logging statement during import error (#274)

Wyświetl plik

@ -0,0 +1 @@
Broken search bar on library home (#278)

Wyświetl plik

@ -29,9 +29,9 @@
</template> </template>
<script> <script>
import _ from 'lodash'
import axios from 'axios' import axios from 'axios'
import logger from '@/logging' import logger from '@/logging'
import backend from '@/audio/backend'
import AlbumCard from '@/components/audio/album/Card' import AlbumCard from '@/components/audio/album/Card'
import ArtistCard from '@/components/audio/artist/Card' import ArtistCard from '@/components/audio/artist/Card'
@ -50,7 +50,6 @@ export default {
albums: [], albums: [],
artists: [] artists: []
}, },
backend: backend,
isLoading: false isLoading: false
} }
}, },
@ -61,7 +60,7 @@ export default {
this.search() this.search()
}, },
methods: { methods: {
search () { search: _.debounce(function () {
if (this.query.length < 1) { if (this.query.length < 1) {
return return
} }
@ -77,15 +76,11 @@ export default {
self.results = self.castResults(response.data) self.results = self.castResults(response.data)
self.isLoading = false self.isLoading = false
}) })
}, }, 500),
castResults (results) { castResults (results) {
return { return {
albums: results.albums.map((album) => { albums: results.albums,
return backend.Album.clean(album) artists: results.artists
}),
artists: results.artists.map((artist) => {
return backend.Artist.clean(artist)
})
} }
} }
}, },