Creating a new post will overwrite an existing post when the file already exists

[alzeih]

Description:

Creating a new post will overwrite an existing post sometimes.

This happens when the filename where the post is to be saved is the same as an existing filename.

Tell us a bit about yourself:

• Version of JekyllAdmin I'm using <HINT: use bundle show to check>: jekyll-admin (0.8.1)
• Version of Jekyll I'm using <HINT: use bundle show to check>: jekyll (3.8.5)
• Version of NodeJS I'm using <HINT: use node -v to check>: none
• Operating System <e.g. OS X, Windows>: linux
• Browser <e.g, Safari, Chrome>: Chrome

Steps to reproduce:

1. Create a new (first) post with the title as "test" and no path.
2. Add some content to the first post.
3. Create the first post.
4. Then, create another new (second) post also with the title "test" and no path. This should be done close enough in time to the first one, i.e: the same day usually.
5. Add some different content to the second post.
6. Create the second post.

I expected the following:

The second post should not not overwrite the first post.

But got the following, instead:

The first post will be replaced by the second post, and the first post is lost.

Other details:

This isn't quite so bad with posts as the filename usually contains a date, so it's less likely to accidentally happen.

With collections though which the title can be the only element that ends up in the filename, it's easier to overwrite an existing item in a collection. (repeat steps above with a collection other than posts that doesn't have a date in the file name / path.)

Overwriting the file makes sense for editing existing posts, but I don't think it makes sense when creating new posts. (Same for collections.)

[lexoyo]

Hi @alzeih
It's actually quite simple to fix
What is the expected behavior, should we either her number after define name if the file exists?

[mertkahyaoglu]

IMO, adding a number (filename_1) is a good way to solve this issue.

[jekyllbot]

This issue has been automatically marked as stale because it has not been commented on for at least two months.

The resources of the Jekyll team are limited, and so we are asking for your help.

If this is a bug and you can still reproduce this error on the master branch, please reply with all of the information you have about it in order to keep the issue open.

If this is a feature request, please consider whether it can be accomplished in another way. If it cannot, please elaborate on why it is core to this project and why you feel more than 80% of users would find this beneficial.

This issue will automatically be closed in two months if no further activity occurs. Thank you for all your contributions.

[alzeih]

Replying to remove stale label.

Raising an error would be fine by me. I just want to avoid overwriting an existing file.

[jekyllbot]

This issue has been automatically marked as stale because it has not been commented on for at least two months.

The resources of the Jekyll team are limited, and so we are asking for your help.

If this is a bug and you can still reproduce this error on the master branch, please reply with all of the information you have about it in order to keep the issue open.

If this is a feature request, please consider whether it can be accomplished in another way. If it cannot, please elaborate on why it is core to this project and why you feel more than 80% of users would find this beneficial.

This issue will automatically be closed in two months if no further activity occurs. Thank you for all your contributions.