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

Issue 1309 #166

Merged
merged 2 commits into from
Feb 13, 2020
Merged

Issue 1309 #166

merged 2 commits into from
Feb 13, 2020

Conversation

dannylamb
Copy link
Member

GitHub Issue: Islandora/documentation#1309

For testing Islandora/islandora#761

What does this Pull Request do?

Pulls in code for Islandora/islandora#761 so it can be tested.

What's new?

Islandora depending on drupal/transliterate_filenames

How should this be tested?

  • Try to upload a file with a comma or a square bracket in it.
  • Feel the pain and heartbreak because your file gets rejected
  • Pull in this PR
  • vagrant up
  • Try to upload a file with a comma or a square bracket in it.
  • Let your heart fill with joy because your filename got sanitized on the way in, and are now acceptable.

Interested parties

@Islandora-Devops/committers

@seth-shaw-unlv
Copy link
Contributor

See this comment: Islandora/islandora#761 (comment)

@dannylamb dannylamb changed the title Update drupal.yml Issue 1309 Feb 13, 2020
@dannylamb
Copy link
Member Author

@seth-shaw-unlv Ok, I've added the module to the appropriate variables in the webserver role. Islandora/islandora#761 now just adds the module to the "suggests" block. So we "suggest" you use that module, and if you install with our installer... you get it :)

@seth-shaw-unlv
Copy link
Contributor

I can probably spin this one up in a few minutes.

Copy link
Contributor

@seth-shaw-unlv seth-shaw-unlv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works as advertised.

My only concern is that we don't auto-populate the Original Name field and some users might be confused why their filename changed. That said, we can move forward with this so repositories don't break and document why we are doing this.

@seth-shaw-unlv seth-shaw-unlv merged commit d4c4167 into dev Feb 13, 2020
@dannylamb
Copy link
Member Author

Lets make an issue for that if we don't already have one.

@dannylamb dannylamb deleted the issue-1309 branch February 13, 2020 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants