mirror of
https://github.com/jellyfin/jellyfin-web
synced 2025-03-30 19:56:21 +00:00
Fix some more code smells
This commit is contained in:
parent
1b8474b4a3
commit
afee01602f
4 changed files with 4 additions and 4 deletions
|
@ -143,7 +143,7 @@ const UserParentalControl: FunctionComponent = () => {
|
||||||
|
|
||||||
for (const btnDelete of accessScheduleList.querySelectorAll('.btnDelete')) {
|
for (const btnDelete of accessScheduleList.querySelectorAll('.btnDelete')) {
|
||||||
btnDelete.addEventListener('click', function () {
|
btnDelete.addEventListener('click', function () {
|
||||||
const index = parseInt(btnDelete.getAttribute('data-index') || '0', 10);
|
const index = parseInt(btnDelete.getAttribute('data-index') ?? '0', 10);
|
||||||
schedules.splice(index, 1);
|
schedules.splice(index, 1);
|
||||||
const newindex = schedules.filter(function (i: number) {
|
const newindex = schedules.filter(function (i: number) {
|
||||||
return i != index;
|
return i != index;
|
||||||
|
|
|
@ -102,7 +102,7 @@ const UserProfile: FunctionComponent = () => {
|
||||||
const target = evt.target as HTMLInputElement;
|
const target = evt.target as HTMLInputElement;
|
||||||
const file = (target.files as FileList)[0];
|
const file = (target.files as FileList)[0];
|
||||||
|
|
||||||
if (!file || !file.type.match('image.*')) {
|
if (!/image.*/.exec(file?.type)) {
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -31,7 +31,7 @@ const ServerContentPage: FunctionComponent<ServerContentPageProps> = ({ view })
|
||||||
|
|
||||||
viewManager.tryRestoreView(viewOptions)
|
viewManager.tryRestoreView(viewOptions)
|
||||||
.catch(async (result?: RestoreViewFailResponse) => {
|
.catch(async (result?: RestoreViewFailResponse) => {
|
||||||
if (!result || !result.cancelled) {
|
if (!result?.cancelled) {
|
||||||
const apiClient = ServerConnections.currentApiClient();
|
const apiClient = ServerConnections.currentApiClient();
|
||||||
|
|
||||||
// Fetch the view html from the server and translate it
|
// Fetch the view html from the server and translate it
|
||||||
|
|
|
@ -179,7 +179,7 @@ export class AlphaPicker {
|
||||||
|
|
||||||
if (item) {
|
if (item) {
|
||||||
const prefix = item.getAttribute('data-prefix');
|
const prefix = item.getAttribute('data-prefix');
|
||||||
if (prefix && prefix.length) {
|
if (prefix?.length) {
|
||||||
itemFocusValue = prefix[0];
|
itemFocusValue = prefix[0];
|
||||||
if (itemFocusTimeout) {
|
if (itemFocusTimeout) {
|
||||||
clearTimeout(itemFocusTimeout);
|
clearTimeout(itemFocusTimeout);
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue