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

create a list as copy does not copy all configuration (template files) #1290

Closed
igoltz opened this issue Dec 6, 2021 · 1 comment · Fixed by #1289
Closed

create a list as copy does not copy all configuration (template files) #1290

igoltz opened this issue Dec 6, 2021 · 1 comment · Fixed by #1289
Labels
Milestone

Comments

@igoltz
Copy link

igoltz commented Dec 6, 2021

Creating a new list as copy of an existing one then this does not

  1. copy multipart mime footer/header (*.mime files)
  2. copy template directories

Version

probably all new, seen in 6.2.64

Installation method

from source package

Expected behavior

If creating a new list as copy of an existing one then this should also

  1. copy multipart mime footer/header (*.mime files)
  2. copy template directories

Actual behavior

Creating a new list as copy of an existing one then this does not

  1. copy multipart mime footer/header (*.mime files)
  2. copy template directories

Additional information

See PR #1289 for a simple fix
The fix (PR) does this by adding both *.mime files to the list of file.
And by fixing the copy logic to also copy template directories.

@ikedas ikedas added the bug label Dec 6, 2021
@ikedas ikedas added this to the 6.2.68 milestone Dec 6, 2021
ikedas added a commit that referenced this issue Dec 7, 2021
… igoltz and ikedas

Creating a list as copy does not copy all configuration (template files etc.) (#1290)
@ikedas
Copy link
Member

ikedas commented Dec 7, 2021

@igoltz , your fix has been merged. Thank you for reporting and fixing the bug!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants