View Issue Details

IDProjectCategoryView StatusLast Update
0003539HTML & PERLFeature Request - Databasepublic2022-11-28 15:26
ReporterCDB-Man Assigned To 
PrioritynormalSeverityminorReproducibilityN/A
Status newResolutionopen 
Summary0003539: Add song-anime-rel comments
DescriptionSimilar to how char-char-rel allow comments, add comments to song-anime-rel.

Example: https://anidb.net/song/108675/anime

Songs have various issues that need to be documented, for example:
1. credited vs uncredited
2. timestamp info to help with song crediting
3. different cuts of songs
4. other idiosyncrasies where having the ability to document comments would be helpful

Currently, since song-anime-rel do not have a comment field, the comments are being stored in the song entries themselves. The problem with this is that anime-relation-specific data is being stored in the song entry. For example, if an insert song is used across 3 anime in the franchise, then comments entered in the song description for anime 1 will not apply to anime 2 and 3.

Further discussion was had in here: https://anidb.net/admin/creq/16411850
TagsNo tags attached.

Activities

Zabr

2022-11-28 15:26

reporter   ~0004483

As I asked in https://anidb.net/admin/creq/16411850 and saw no objection, the only data we want to have and maintain in anime-song-rel is broadcast type and timestamps.

The first suggestion is:
1 new column to anime-song table:
- description (text)

In this text field we will fill info by the template, if necessary.

This one will be easier and faster to implement, but I think it will create next problems in the future:
- it's just unvalidated text field and anybody can write here anything;
- even if we will follow the template, it easily could be mistakes, typos, wrong separators;
- as ADD creqs process instantly, nobody can notice the errors mentioned above and fix them right away.

The second suggestion is:
2 new columns to anime-song table:
- broadcast_type (enum)
- timestamps (jsonb) (json column should works good there, because we only need to save and read this data without any filtering)

timestamps format in the attached file (timestamps is array to support multiply plays of the 1 track in the 1 episode)

All data could be validated:
- broadcast_type - is final list of options
- episode_id / episode_number - should be in this relation episode field;
- timestamp - should be time.

This one will be harder and slower to implenent, but because of validation, will be easier to maintain.

The only question is front view (anime detail, song detail, add/edit forms).
Also, as we will have episode in json, this data could be used (shown) on file detail pages for certain episode.
timestamp.json (195 bytes)   
{
    "episode_id_1/episode_number_1": [
        "[h]:mm:ss",
        "mm:ss",
        "m:ss",
        ...
    ],
    "episode_id_2/episode_number_2": [
        "m:ss"
    ],
    ...
}
timestamp.json (195 bytes)   

Issue History

Date Modified Username Field Change
2022-11-21 16:17 CDB-Man New Issue
2022-11-28 15:26 Zabr Note Added: 0004483
2022-11-28 15:26 Zabr File Added: timestamp.json