Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

COOL_SEARCH_LINKS: Show pure entity edits (with and without relationships) #664

Open
jesus2099 opened this issue Jan 26, 2022 · 12 comments
Open

Comments

@jesus2099
Copy link
Owner

jesus2099 commented Jan 26, 2022

https://tickets.metabrainz.org/browse/MBS-11374

Filter out:

  • recording edits
  • work edits
jesus2099 added a commit that referenced this issue May 3, 2022
In artist, release and release group sidebar #664
jesus2099 added a commit that referenced this issue May 3, 2022
@jesus2099
Copy link
Owner Author

I did not include relationships, though.
I don't know if it is possible.

@jesus2099 jesus2099 reopened this May 3, 2022
@jesus2099
Copy link
Owner Author

It should work with does not match any and release is not MBID and type is not xxx and Relationship type is not yyy.
But it didn't work!

Example, this release has 2 Add disc ID edits.

The same release has 1 discogs edit.

But if I combine these (very long search time): I only get the disc ID edits, discogs edit is missing.

@jesus2099
Copy link
Owner Author

Ah OK it works but I have to include Add relationship in the edit types!

@jesus2099
Copy link
Owner Author

So maybe I will do this (and Remove relationship, and Add/Remove relationship attribute?) but it makes a very slow search to try to have both non-relationship edits and relationship edits.

@jesus2099
Copy link
Owner Author

jesus2099 commented May 3, 2022

I will make this:

Example with a release:

@jesus2099
Copy link
Owner Author

jesus2099 commented May 3, 2022

Maybe I should try a more top-down logic order, where we can see which ones are included in which?

Edit amounts are here for illustration, it would be too naughty with MBS to display them.

@jesus2099
Copy link
Owner Author

There are 3 Add relationship (post-NGS) edits (type 90):

But also 5 Add relationship (pre‐NGS) edits (type 233):

Pre-NGS edit types are incompatible with post-NGS relationship types.
So if I want an edit search that gives me all relationship edits, I must use the same technique (edits that do not match any of the following conditions: is not, is not, is not) as the full search, which is very slow.

@jesus2099
Copy link
Owner Author

jesus2099 commented May 3, 2022

I could make a quick AJAX / XHR request to test if this entity has any pre-NGS relationship edits, to switch to the quick query, if not, and keep the slow query, if yes (if the test returns <div id="edits">).

Or maybe even more clever, is the NGS frontier the same as these old/new relationship frontier?

Maybe I can use this 14459456 NGS magic number, or its date (2011-05-17), to avoid an AJAX / XHR check?

Maybe I could try to find the first NGS entity row ID for each type! No it's not valid.

Maybe those row ID could be reliable to judge if I put the slow search (pre-NGS entities) or the quick search (post-NGS entities).

But my method is not valid.
A newer row ID could still be pre-NGS if it was merged into a newer entity (greater row ID).

jesus2099 added a commit that referenced this issue May 3, 2022
@jesus2099 jesus2099 changed the title COOL_SEARCH_LINKS: Show release edits only COOL_SEARCH_LINKS: Show pure entity releaedits only May 3, 2022
@jesus2099 jesus2099 changed the title COOL_SEARCH_LINKS: Show pure entity releaedits only COOL_SEARCH_LINKS: Show pure entity edits (with and without relationships) May 3, 2022
@jesus2099
Copy link
Owner Author

jesus2099 commented May 3, 2022

IMG_20220503_184900
Kiwi Browser in landscape mode

Strange display on mobile, both in portrait and landscape modes.
The Pure entity edits should not appeared between refine and effective.

Maybe it's because the script running order is not the same in my mobile and on my PC.

@jesus2099
Copy link
Owner Author

It's indeed a problem in PENDING EDITS.

IMG_20220503_195032

As soon as I run it before COOL_SEARCH_LINKS, it displays nicely.

jesus2099 added a commit that referenced this issue May 3, 2022
#664 (comment)
Now works nicely, whatever the script running order
@jesus2099
Copy link
Owner Author

Apparently I forgot Add medium edits for Releases.

jesus2099 added a commit that referenced this issue May 8, 2022
In artist, release and release group sidebar #664
jesus2099 added a commit that referenced this issue May 8, 2022
jesus2099 added a commit that referenced this issue May 8, 2022
jesus2099 added a commit that referenced this issue May 8, 2022
#664 (comment)
Now works nicely, whatever the script running order
jesus2099 added a commit that referenced this issue May 13, 2022
jesus2099 added a commit that referenced this issue May 27, 2022
jesus2099 added a commit that referenced this issue May 29, 2022
Make an ajax check search for pre-NGS relationship
jesus2099 added a commit that referenced this issue May 30, 2022
Improve display when pre‐NGS relationship edits (not) found
@jesus2099
Copy link
Owner Author

Apparently I forgot to add links for Pure recording edits: https://musicbrainz.org/recording/2e87df3a-7b74-4ee3-8d04-73dfb094ed9f

@jesus2099 jesus2099 reopened this Mar 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant