Saved Searches properties GUI - insufficient names and query validation about gtg HOT 4 OPEN

Regarding point no.3, I believe the current code validated for non-empty names, but not non-empty queries ; in think it should also check for that (and ideally do some more advanced checks for valid syntax or something, but I don't know exactly what criteria to apply there)

from gtg.

For your problem no2, I bet there is some relationship between this and #1005 (for tags)...

from gtg.

Regarding 2, if I add explicit definition for ds.saved_searches.remove, the problem is reverse - search is deleted from the list immediately, but after restart it returns (so it's not saved probably):

def remove(self, search_id: uuid4) -> None:
    """Remove saved search"""
  
    search = self.lookup[search_id]
    self.data.remove(search)
    pos = self.model.find(search)
    self.model.remove(pos[1])

I'm not a coder, so this might be broken in many ways, but this is what I thought might work at least. It's just some input which could help debug the problem.

from gtg.

Related: #1048

from gtg.