View Issue Details

IDProjectCategoryView StatusLast Update
0003616Composr alpha bug reports[All Projects] General / Uncategorisedpublic2018-07-20 05:49
ReporterChris GrahamAssigned ToSalman 
SeverityFeature-request 
Status resolvedResolutionfixed 
Summary0003616: Broken uploads
DescriptionAs discussed on Skype
Additional InformationSalman, 17:46
ok (y) just found some instances I missed when hyphenating id attributes

Salman, 17:52
click_link(document.getElementById('uploadButton_file'+num_attachments));

is num_attachments a local variable or global aka window.num_attachments?

oh looks like the latter

Salman, 18:33
the submit_form_on_enter stuff was easy, just add data-submit-on-enter="1" to the desired elements haha




click_link(document.getElementById('uploadButton_file'+num_attachments));

^ is document.getElementById('uploadButton_file'+num_attachments) a button or a link?

it won't work for a button as it looks like I replaced click_link() with $util.navigate(), which doesn't actually trigger a click event anymore rather only redirects to the value of the [href] attribute

19:46
I believe uploadButton_file is a button created dynamically in plupload.js
        var settings = getUploaderSettings(name, pageType, postingFieldName, filter);
        settings.progress_target = 'fs-upload-progress-' + name;
        settings.required = rep.classList.contains('required');
        settings.browse_button = 'upload-button-' + name;
        settings.drop_element = 'txt_filename_' + name;
        settings.container = 'main-div-' + name;
there it is, yes inconsistent

Salman, 19:50
Ok I will check it after you commit.
TagsNo tags attached.
Sponsorship open0

Activities

Salman

2018-07-19 06:01

reporter   ~0005769

Should be fixed now. I've replaced the $util.navigate() call with a click_link() equivalent for non-links: $dom.trigger(element, 'click'). Couldn't test it unfortunately as I wasn't able to find out how to make the attachments UI show up, in a reasonable amount of time. Can you test it or give pointers?

https://github.com/ocproducts/composr/commit/bfa00849a0ea31ceddeca96d39ab1ce5b3977aea

Salman

2018-07-20 05:49

reporter   ~0005771

Ah, it shows up with simplified attachments UI disabled, tested working now!

Issue History

Date Modified Username Field Change
2018-05-28 01:15 Chris Graham New Issue
2018-05-28 01:15 Chris Graham Status non-assigned => assigned
2018-05-28 01:15 Chris Graham Assigned To => Salman
2018-07-19 06:01 Salman Note Added: 0005769
2018-07-20 05:49 Salman Note Added: 0005771
2018-07-20 05:49 Salman Status assigned => resolved
2018-07-20 05:49 Salman Resolution open => fixed
2023-02-26 18:29 Chris Graham Category General => General / Uncategorised