refactor(media-request): remove excessive debug logging
This commit is contained in:
@@ -332,14 +332,6 @@ export class MediaRequest {
|
||||
if (requestBody.mediaType === MediaType.MOVIE) {
|
||||
await mediaRepository.save(media);
|
||||
|
||||
logger.debug('Media saved', {
|
||||
label: 'Media Request',
|
||||
mediaType: MediaType.MOVIE,
|
||||
mediaId: media.id,
|
||||
mediaTmdbId: media.tmdbId,
|
||||
mediaHasId: !!media.id,
|
||||
});
|
||||
|
||||
const request = new MediaRequest({
|
||||
type: MediaType.MOVIE,
|
||||
media,
|
||||
@@ -381,43 +373,7 @@ export class MediaRequest {
|
||||
isAutoRequest: options.isAutoRequest ?? false,
|
||||
});
|
||||
|
||||
logger.debug('Request object before save', {
|
||||
label: 'Media Request',
|
||||
mediaType: MediaType.MOVIE,
|
||||
hasMediaObject: !!request.media,
|
||||
mediaObjectId: request.media?.id,
|
||||
requestMediaIdProperty: (request as any).mediaId,
|
||||
requestKeys: Object.keys(request),
|
||||
requestType: request.type,
|
||||
requestStatus: request.status,
|
||||
requestIs4k: request.is4k,
|
||||
requestServerId: request.serverId,
|
||||
requestProfileId: request.profileId,
|
||||
});
|
||||
|
||||
await requestRepository.save(request);
|
||||
|
||||
logger.debug('Request saved', {
|
||||
label: 'Media Request',
|
||||
mediaType: MediaType.MOVIE,
|
||||
requestId: request.id,
|
||||
requestMediaIdAfterSave: (request as any).mediaId,
|
||||
});
|
||||
|
||||
const rawResult = await requestRepository.query(
|
||||
'SELECT id, "mediaId", type, status, "createdAt" FROM media_request WHERE id = $1',
|
||||
[request.id]
|
||||
);
|
||||
|
||||
logger.debug('Request verified from DB (raw query)', {
|
||||
label: 'Media Request',
|
||||
mediaType: MediaType.MOVIE,
|
||||
requestId: request.id,
|
||||
rawDbResult: rawResult[0],
|
||||
mediaIdInDB: rawResult[0]?.mediaId,
|
||||
mediaIdIsNull: rawResult[0]?.mediaId === null,
|
||||
});
|
||||
|
||||
return request;
|
||||
} else {
|
||||
const tmdbMediaShow = tmdbMedia as Awaited<
|
||||
@@ -486,14 +442,6 @@ export class MediaRequest {
|
||||
|
||||
await mediaRepository.save(media);
|
||||
|
||||
logger.debug('Media saved', {
|
||||
label: 'Media Request',
|
||||
mediaId: media.id,
|
||||
mediaType: MediaType.TV,
|
||||
mediaTmdbId: media.tmdbId,
|
||||
mediaHasId: !!media.id,
|
||||
});
|
||||
|
||||
const request = new MediaRequest({
|
||||
type: MediaType.TV,
|
||||
media,
|
||||
@@ -556,42 +504,7 @@ export class MediaRequest {
|
||||
isAutoRequest: options.isAutoRequest ?? false,
|
||||
});
|
||||
|
||||
logger.debug('Request object before save', {
|
||||
label: 'Media Request',
|
||||
mediaType: MediaType.TV,
|
||||
hasMediaObject: !!request.media,
|
||||
mediaObjectId: request.media?.id,
|
||||
requestMediaIdProperty: (request as any).mediaId,
|
||||
requestKeys: Object.keys(request),
|
||||
seasonCount: request.seasons.length,
|
||||
requestStatus: request.status,
|
||||
requestServerId: request.serverId,
|
||||
requestProfileId: request.profileId,
|
||||
});
|
||||
|
||||
await requestRepository.save(request);
|
||||
|
||||
logger.debug('Request saved', {
|
||||
label: 'Media Request',
|
||||
mediaType: MediaType.TV,
|
||||
requestId: request.id,
|
||||
requestMediaIdAfterSave: (request as any).mediaId,
|
||||
});
|
||||
|
||||
const rawResult = await requestRepository.query(
|
||||
'SELECT id, "mediaId", type, status, "createdAt" FROM media_request WHERE id = $1',
|
||||
[request.id]
|
||||
);
|
||||
|
||||
logger.debug('Request verified from DB (raw query)', {
|
||||
label: 'Media Request',
|
||||
mediaType: MediaType.TV,
|
||||
requestId: request.id,
|
||||
rawDbResult: rawResult[0],
|
||||
mediaIdInDB: rawResult[0]?.mediaId,
|
||||
mediaIdIsNull: rawResult[0]?.mediaId === null,
|
||||
});
|
||||
|
||||
return request;
|
||||
}
|
||||
}
|
||||
|
||||
@@ -780,16 +780,6 @@ export class MediaRequestSubscriber implements EntitySubscriberInterface<MediaRe
|
||||
media[statusKey] !== MediaStatus.PARTIALLY_AVAILABLE &&
|
||||
media[statusKey] !== MediaStatus.PROCESSING
|
||||
) {
|
||||
logger.debug(
|
||||
'updateParentStatus: updating media status to PROCESSING via direct update',
|
||||
{
|
||||
label: 'Media Request',
|
||||
requestId: entity.id,
|
||||
mediaId: media.id,
|
||||
statusKey,
|
||||
previousStatus: media[statusKey],
|
||||
}
|
||||
);
|
||||
media[statusKey] = MediaStatus.PROCESSING;
|
||||
await mediaRepository.save(media);
|
||||
}
|
||||
@@ -799,14 +789,6 @@ export class MediaRequestSubscriber implements EntitySubscriberInterface<MediaRe
|
||||
entity.status === MediaRequestStatus.DECLINED &&
|
||||
media[statusKey] !== MediaStatus.DELETED
|
||||
) {
|
||||
logger.debug(
|
||||
'updateParentStatus: setting declined movie status to UNKNOWN',
|
||||
{
|
||||
label: 'Media Request',
|
||||
requestId: entity.id,
|
||||
mediaId: media.id,
|
||||
}
|
||||
);
|
||||
media[statusKey] = MediaStatus.UNKNOWN;
|
||||
await mediaRepository.save(media);
|
||||
}
|
||||
@@ -827,13 +809,6 @@ export class MediaRequestSubscriber implements EntitySubscriberInterface<MediaRe
|
||||
where: { media: { id: media.id }, status: MediaRequestStatus.PENDING },
|
||||
});
|
||||
|
||||
logger.debug('updateParentStatus: TV declined check', {
|
||||
label: 'Media Request',
|
||||
requestId: entity.id,
|
||||
mediaId: media.id,
|
||||
pendingCount,
|
||||
});
|
||||
|
||||
if (pendingCount === 0) {
|
||||
// Re-fetch media without requests to avoid cascade issues
|
||||
const freshMedia = await mediaRepository.findOne({
|
||||
@@ -852,12 +827,6 @@ export class MediaRequestSubscriber implements EntitySubscriberInterface<MediaRe
|
||||
entity.status === MediaRequestStatus.APPROVED
|
||||
) {
|
||||
entity.seasons.forEach((season) => {
|
||||
logger.debug('updateParentStatus: approving season request', {
|
||||
label: 'Media Request',
|
||||
requestId: entity.id,
|
||||
seasonRequestId: season.id,
|
||||
seasonNumber: season.seasonNumber,
|
||||
});
|
||||
season.status = MediaRequestStatus.APPROVED;
|
||||
seasonRequestRepository.save(season);
|
||||
});
|
||||
@@ -890,19 +859,9 @@ export class MediaRequestSubscriber implements EntitySubscriberInterface<MediaRe
|
||||
});
|
||||
|
||||
if (needsStatusUpdate) {
|
||||
logger.debug('handleRemoveParentUpdate: setting status to UNKNOWN', {
|
||||
label: 'Media Request',
|
||||
requestId: entity.id,
|
||||
mediaId: cleanMedia.id,
|
||||
});
|
||||
cleanMedia.status = MediaStatus.UNKNOWN;
|
||||
}
|
||||
if (needs4kStatusUpdate) {
|
||||
logger.debug('handleRemoveParentUpdate: setting status4k to UNKNOWN', {
|
||||
label: 'Media Request',
|
||||
requestId: entity.id,
|
||||
mediaId: cleanMedia.id,
|
||||
});
|
||||
cleanMedia.status4k = MediaStatus.UNKNOWN;
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user