Journal entry add file should be a button

[Liam]

Journal entry added a file: Shouldn't the "Remove" link be a button?

[Mahara Bot (dev-mahara)]

Patch for "master" branch: https://reviews.mahara.org/5694

[Aaron Wells (u-aaronw)]

To test:

1. Create a new journal entry.
2. Click the "Add a file" button and upload or choose a file in the modal. Then close the modal by clicking the close icon in the upper-right.
Expected: You should see the file in the list of attachments, with a red "x" icon and the word "Remove" styled as if they're in a button.
3. Save the journal entry.
4. On the list of journal entries, click the "edit" button to go back to editing the journal entry you just saved.
5. Look at the list of attachments.
Expected: You should again see the file listed, with a red "x" icon and the word "Remove" styled as if in a button.

The "add new" and "edit existing" test cases have to be tested separately, because the "add new" case is adding a button which is dynamically generated using javascript, and the "edit existing" is looking a button that is rendered from a .tpl file. Ideally, these two should create exactly the same HTML. (Perhaps we ought to refactor this so that they use the same code to render both somehow?)

[Dean (deans)]

When I tested this I found some strange behaviour by mistake.

Steps to reproduce:
- create journal
- Make sure you leave the Entry * field empty
- Click the "Add a file" button and upload or choose a file in the modal
- Attempt to save the page
- Error message displays saying that you need something in the Entry* field to be able to save
- Remove the file that was uploaded and click on 'add a file' button again to upload another file

Expected result:
The upload file screen should display as it did the first time you uploaded the file

Actual result:
Upload screen is not displaying correctly and is missing information

[Liam (liam-sharpe)]

I had a look around and managed to reproduce the bug but I think it is more of a php/backend issue so im not to sure on how to fix it

[Mahara Bot (dev-mahara)]

Reviewed: https://reviews.mahara.org/5694
Committed: https://git.mahara.org/mahara/mahara/commit/17190a621de9a3da851a7d6de4e3cf4cbb7e91c0
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 17190a621de9a3da851a7d6de4e3cf4cbb7e91c0
Author: Liam Sharpe <email address hidden>
Date: Thu Nov 12 10:06:40 2015 +1300

Journal add file remove is now a button - Bug #1515398

- Fixed 'Remove' button for existing attachments
- Fixed reloading of modal popup for 'Add a file' if form reloads with
error

behatnotneeded

Change-Id: Ib68e6a33beb6f85701281fcdf337fafd079c9528

[Mahara Bot (dev-mahara)]

Patch for "16.04_STABLE" branch: https://reviews.mahara.org/6360

[Mahara Bot (dev-mahara)]

Reviewed: https://reviews.mahara.org/6360
Committed: https://git.mahara.org/mahara/mahara/commit/60481797cc189f53dd804b4295640c34eb930544
Submitter: Robert Lyon (<email address hidden>)
Branch: 16.04_STABLE

commit 60481797cc189f53dd804b4295640c34eb930544
Author: Liam Sharpe <email address hidden>
Date: Thu Nov 12 10:06:40 2015 +1300

Journal add file remove is now a button - Bug #1515398

- Fixed 'Remove' button for existing attachments
- Fixed reloading of modal popup for 'Add a file' if form reloads with
error

behatnotneeded

Change-Id: Ib68e6a33beb6f85701281fcdf337fafd079c9528
(cherry picked from commit 17190a621de9a3da851a7d6de4e3cf4cbb7e91c0)