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
Upload filenames with spaces. #5905
Comments
|
do you mean that array('#(.){2,}#', '#[^A-Za-z0-9._- ]#', '#^.#'); removes spaces? I think not. Are you shore? |
Ah, misunderstood you then. Ignore my comment. You are right, |
I don't really understand the issue here @yozmag. You said that you're having problems with files having spaces in the filename. But, the media manager doesn't allow the user to upload files with spaces in the filename (I just tried to do that and Joomla gave me a notice that the filename shouldn't have any spaces). So, how are you getting these files (with spaces in the filename) in the first place? Are you uploading them manually? Anyway, I suppose, we could allow for spaces in the filename. We could either remove the spaces altogether or replace them with underscores or dashes. This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5905. |
I should have identifyed the place where this append, I'm on joomla 2.5.28, and this file is the same on 3.3. |
I just double checked with 3.4beta1 On 30 January 2015 at 14:33, yozmag notifications@github.com wrote:
Brian Teeman |
I just found this, 3d1af35 . |
@yozmag it was 3.2.0. |
Thank you. |
I am have some problems with og:image because of uploaded images with space in the filename. So could we change public static function makeSafe($file) from $regex = array('#(.){2,}#', '#[^A-Za-z0-9._- ]#', '#^.#'); to $regex = array('#(.){2,}#', '#[^A-Za-z0-9._- ]#', '#^.#','#\s#'); this would solve the problem.
The text was updated successfully, but these errors were encountered: