Fix single_reference=True

[pllim]

Fix #42

[pllim]

Nope does not appear to work.

[ConorMacBride]

I think this should work:

        if filename is None:
            if single_reference:
                filename = item.function + '.' + extension
            else:
                filename = item.name + '.' + extension
                filename = filename.replace('[', '_').replace(']', '_')
                filename = filename.replace('_.' + extension, '.' + extension)

[pllim]

Ah, thanks! I updated the patch and added you as co-author of the commit.

[pllim]

Yay, CI is green now. Thanks!

I have no expertise nor interest to add a unit test but @astrofrog is welcome to push to my branch if he wants a test. 😉

[astrofrog]

I've added a regression test, which unfortunately shows that the fix here doesn't work

[astrofrog]

@ConorMacBride - for some reason the function name is ending up being wrapper so this doesn't work quite right.

[astrofrog]

Ok I got it to work, it was item.originalname we wanted - I'll go ahead and merge and release once CI passes, since this is needed for reproject.

[pllim]

Thanks, @astrofrog !