In IMCE there's the default behaviour for uploading files, wich works well but it's better to let the user decide what to do with the files. Therefor I've created a patch that will add an extra option to the default behaviour in the IMCE settings. When this last option is selected, the user can choose what to do when uploading files.

CommentFileSizeAuthor
#8 interdiff.txt570 bytespaulocs
#8 imce-Let_the_user_decide_wich_action_to_apply_on_uploading-2231251-d7-8.patch2.43 KBpaulocs
#7 2231251-7.patch2.43 KBsuresh prabhu parkala
#3 imce-behaviourbyuser-2231251-8642885.patch2.96 KBnico.knaepen
#1 2231251-let-user-decide-behaviour.patch2.88 KBnico.knaepen

Comments

nico.knaepen’s picture

nico.knaepen commented
StatusFileSize
new 2231251-let-user-decide-behaviour.patch2.88 KB
nico.knaepen’s picture

nico.knaepen commented
Issue summary: View changes
nico.knaepen’s picture

nico.knaepen commented
Status: Active » Needs review
StatusFileSize
new imce-behaviourbyuser-2231251-8642885.patch2.96 KB

I've added a patch for this functionality

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Assigned: nico.knaepen » Unassigned

@nico.knaepen: Thanks for the patch. Please note that you should only set the "assigned" field to your name if you plan on working on something, otherwise leave it unassigned.

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented
Status: Needs review » Needs work
thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented

Needs reroll

suresh prabhu parkala’s picture

suresh prabhu parkala commented
Status: Needs work » Needs review
StatusFileSize
new 2231251-7.patch2.43 KB

Please review!

paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented
Version: 7.x-1.8 » 7.x-1.x-dev
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new imce-Let_the_user_decide_wich_action_to_apply_on_uploading-2231251-d7-8.patch2.43 KB
new interdiff.txt570 bytes

Hello,
I tested the patch #7 and it works good. Now if the common permission is set to "Let the user decide which option to choose", the user can choose what he wants to do with the file. I just did a english correction. I changed "wich" to "which" so I'll let as RTBC

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented

Looks good to me!
Thanks @all!

  • thalles committed aa79038 on 7.x-1.x authored by paulocs 
    Issue #2231251 by nico.knaepen, paulocs, Suresh Prabhu Parkala: Let the...
thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

vistree’s picture

vistree commented

Has this patch been removed from D8 version again?