on the old ticket and pull request for this, you said I broke compatibility
with other things with my change, but I didn’t know what those things were. https://issues.jboss.org/browse/RAILO-2468
I have never experienced a problem from that change, but I’m sure you are
right.
I just upgraded from Railo to Lucee for the first time, and learned how to
install / build it from the bitbucket instructions (you made it easier now
good work), and I did this patch again, and again it fixes the multiple
upload feature.
On the master branch of lucee on bitbucket, I found another way to ensure
the key on that fileItems object is a unique name.
Is this an acceptable fix now?
\lucee-git\lucee-java\lucee-core\src\lucee\runtime\type\scope\FormImpl.java
line 1: 184: change:
fileItems.put(item.getFieldName().toLowerCase(),
to:
fileItems.put(getFileName(),
Micha, I would like this fixed as well, as I think I commented on that
ticket many times. Also I think there might be an inherent slowness with
file upload in Lucee, compared to ACF, I have a setup now where I can test
this, so I will let you know, but maybe the whole thing needs looking into
more closely or even rewriting completely.
on the old ticket and pull request for this, you said I broke
compatibility with other things with my change, but I didn’t know what
those things were. https://issues.jboss.org/browse/RAILO-2468
I have never experienced a problem from that change, but I’m sure you are
right.
I just upgraded from Railo to Lucee for the first time, and learned how to
install / build it from the bitbucket instructions (you made it easier now
good work), and I did this patch again, and again it fixes the multiple
upload feature.
On the master branch of lucee on bitbucket, I found another way to ensure
the key on that fileItems object is a unique name.
Is this an acceptable fix now?
\lucee-git\lucee-java\lucee-core\src\lucee\runtime\type\scope\FormImpl.java
line 1: 184: change:
fileItems.put(item.getFieldName().toLowerCase(),
to:
fileItems.put(getFileName(),
–
See Lucee at CFCamp Oct 22 & 23 2015 @ Munich Airport, Germany - Get your
ticket NOW - http://www.cfcamp.org/
Michael posted a commit for multiple file upload support in the newest
version of 4.5 and 5 on github today.
On Saturday, August 8, 2015 at 3:00:24 PM UTC-4, Andrew Dixon wrote:
Micha, I would like this fixed as well, as I think I commented on that
ticket many times. Also I think there might be an inherent slowness with
file upload in Lucee, compared to ACF, I have a setup now where I can test
this, so I will let you know, but maybe the whole thing needs looking into
more closely or even rewriting completely.
on the old ticket and pull request for this, you said I broke
compatibility with other things with my change, but I didn’t know what
those things were. https://issues.jboss.org/browse/RAILO-2468
I have never experienced a problem from that change, but I’m sure you
are right.
I just upgraded from Railo to Lucee for the first time, and learned how
to install / build it from the bitbucket instructions (you made it easier
now - good work), and I did this patch again, and again it fixes the
multiple upload feature.
On the master branch of lucee on bitbucket, I found another way to
ensure the key on that fileItems object is a unique name.
Is this an acceptable fix now?
\lucee-git\lucee-java\lucee-core\src\lucee\runtime\type\scope\FormImpl.java
line 1: 184: change:
fileItems.put(item.getFieldName().toLowerCase(),
to:
fileItems.put(getFileName(),
–
See Lucee at CFCamp Oct 22 & 23 2015 @ Munich Airport, Germany - Get
your ticket NOW - http://www.cfcamp.org/
Michael posted a commit for multiple file upload support in the newest
version of 4.5 and 5 on github today.On Saturday, August 8, 2015 at 3:00:24 PM UTC-4, Andrew Dixon wrote:
Micha, I would like this fixed as well, as I think I commented on that
ticket many times. Also I think there might be an inherent slowness with
file upload in Lucee, compared to ACF, I have a setup now where I can test
this, so I will let you know, but maybe the whole thing needs looking into
more closely or even rewriting completely.
on the old ticket and pull request for this, you said I broke
compatibility with other things with my change, but I didn’t know what
those things were. https://issues.jboss.org/browse/RAILO-2468
I have never experienced a problem from that change, but I’m sure you are
right.
I just upgraded from Railo to Lucee for the first time, and learned how
to install / build it from the bitbucket instructions (you made it easier
now - good work), and I did this patch again, and again it fixes the
multiple upload feature.
On the master branch of lucee on bitbucket, I found another way to ensure
the key on that fileItems object is a unique name.
Is this an acceptable fix now?
\lucee-git\lucee-java\lucee-core\src\lucee\runtime\type\scope\FormImpl.java
line 1: 184: change:
fileItems.put(item.getFieldName().toLowerCase(),
to:
fileItems.put(getFileName(),
–
See Lucee at CFCamp Oct 22 & 23 2015 @ Munich Airport, Germany - Get your
ticket NOW - http://www.cfcamp.org/
