On 2016-10-12 23:41, Daniel Axtens wrote:
Daniel Axtens <d...@axtens.net> writes:
- "untitled series #420" is certainly better than having every
series
named
"untitled series", but I still think it's worse than "series
starting
with
'[1/10] add foo to bar'"
Should be trivial enough change. I wonder if we should save this into
the db on calculate it on the fly like I do for 'untitled series'?
I'll
need to benchmark it and if the former is no slower, then this can be
a
follow-up patch. If the latter is more performant, we'll need to
respin.
So this is the only one I'm at all concerned about potentially needing
a
respin.
You'd have to find the first patch in the series to work it out. I'm a
bit nervous that would be expensive.
Oh, one potential complexity - do we handle the case where the patches
arrive with correct headers but are received out of order? In that
case,
this is a pretty tricky problem.
I'm thinking something like this:
def add_cover(cover):
...
if not series.name or series.name.startswith('Series starting
with '):
series.name = cover.name.split(']')[:-1]
...
i.e. if the subject was automatically set then we overwrite it with more
recent information. If it was manually set then don't touch it.
Thoughts?
Stephen
_______________________________________________
Patchwork mailing list
Patchwork@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/patchwork