behatnotneeded

Change-Id: I788f932f721b070f188d97891ab21f0f5f77811b
Signed-off-by: Robert Lyon <robertl@catalyst.net.nz>

Add in the new search by and order options

Change-Id: If0ab8397793a3aad3df4d7fa4e4a69ccd4b908b9

Change-Id: I7081b319ff410aceeae01eb9d9f83fba95e99638

Allow to set the theme in view table on the DB only when
  we have userscanchooseviewthemes = true and
  the theme is set when we are editing page

behatnotneeded

Change-Id: I16ef25e8efa6060aa7e5a30a44ed2f4698f6d625

When a journal entry is marked as draft,
    we can't add it to a block
    we don't show it in a page, instead we show a
    placeholder with a message
see bug 1387858

Change-Id: I73c475bbc6baddc0dc38c9e7a2c3c60b24a617b4
behatnotneeded

Cecilia Vela Gurovic authored Feb 20, 2017 and Robert Lyon committed Mar 05, 2017

behatnotneeded

Change-Id: I202828d9fb16ef9c9c0057ad2f03845683e31551

Replaced most calls to DB to request group
basic information, for get_group_by_id() method
in group library

behatnotneeded

Change-Id: Ie03520a6c7a364ead63ccedaff07118a5928348e

option to delete page is enable for:
    - Normal user in its own pages
    - For group pages
        To admin user
        To other roles if they have Create and Edit permission
         in group setting and page is not locked
    - For institution, to site admin

behatnotneeded

Change-Id: I424c8a103e55d46178d779b77ebfb79db2d504f3

Checks if the theme exists before creating
the profile page for a new user

behatnotneeded

Change-Id: Ia3a49588c02032b427be0559a8f9fea98975f0d9

When it is set to be deleted straight away via parameter 'deleted' = true

behatnotneeded

Change-Id: I26204b8d358520c6acf3434531c06c4490152f47
Signed-off-by: Robert Lyon <robertl@catalyst.net.nz>

behatnotneeded

Change-Id: I460917ee97d715b4b7e2c3f1771b87b105fdbae1

Cecilia Vela Gurovic authored Oct 21, 2016 and Robert Lyon committed Nov 14, 2016

In the block Recent journal entries, the drop-down menu
only shows the Journals which the user has permission to
add a new entry.

Also corrected error showing up and broken Journal search
in the Edit block side screen.

Also fixed the poor alignment of the artefactchoooser search form's
search button

Also fixes Bug 1636850 with change in lib/view.php

behatnotneeded

Change-Id: I44c0d6d25eda7cb37d4a8aab66a6d1b93ca60b69

When adding a Recent journal entries block to a page,
the group journals listed to be selected show the group name
as a detail, instead of the group shortname

behatnotneeded

Change-Id: Ib26a72cd353928eaf7255dcb0adb6a626ab53c26

behatnotneeded

Change-Id: I81b1ca00092a4e8bad6807855056bcf79186732e

Currently only sets new page to institution theme rather than what
theme you specified in account preferences.

behatnotneeded - visual difference

Change-Id: Ie3b5f852f2a059f6b7dab1a86f8e8cd3d35015a9
Signed-off-by: Robert Lyon <robertl@catalyst.net.nz>

When a group, institution or site page is copied,
the page description from the template page
should be copied to the new page.

behatnotneeded

Change-Id: I89dd24457005c2b2d43abfbb65824198d85f8fd1

Sometimes in MySql if many users are copying a page at once the
view_rows_columns information can get deleted.

As we can work out what the values should have been from other tables
we can regenerate it and add it back in.

If the view doesn't have a layout set it hasn't changed from when
first been made so we can make it the same as view type's template

To test: see bug report

behatnotneeded

Change-Id: I81c8e33dc1bc4121c1df97e1acbc981ad969271c
Signed-off-by: Robert Lyon <robertl@catalyst.net.nz>

Cecilia Vela Gurovic authored Sep 19, 2016 and Cecilia Vela Gurovic committed Sep 28, 2016

Change group homepage so that it does not to show repeated pages/collections in different sections shared/submitted
Updated create_group() in TestingDataGenerator.php to set in the DB the correct group role of a user

Change-Id: I07504f38a35d4dda4a3c4f440d94cffed293bbf2

with $smarty->assign()
behatnotneeded

Change-Id: I667463b7732bd3f1dd2619b2836cf4b8c560d264

Because certain libraries are not loaded under some circumstances

When copying a collection containing pages with descriptions
containing embedded images to/from a group/user.

behatnotneeded

Change-Id: I1a5f4e3d2c26736b6e087cf07daa5fd44f84441f
Signed-off-by: Robert Lyon <robertl@catalyst.net.nz>

Should be code changes only

behatnotneeded

Change-Id: Ie0a35a8cf54d3f19b6875022a3f6a9f30de1cbd3
Signed-off-by: Robert Lyon <robertl@catalyst.net.nz>

This adds the framework matrix page as the first page of the collection

Change-Id: I229fe2f08c636f5f0a63a9284304c15de454bbfd
Signed-off-by: Robert Lyon <robertl@catalyst.net.nz>

Liam authored Aug 24, 2016 and Robert Lyon committed Aug 24, 2016

Also swapped round the 'copy page' and 'copy collection' buttons

behatnotneeded

Change-Id: Ic0c32364e51b56ffbcf7108877c6ea2820ebf45c

Leaving in View::get_artefact_instances() because there are
plans to use it in the SmartEvidence commits.

Change-Id: I04371779ba358678ae5b36b286356e2fd990dfff
behatnotneeded: Covered by existing tests.

When adding a page to a collection and they have different access rules

Change-Id: Ib45bc551309a592f8863d6ea051f26f07c4797ab
Signed-off-by: Robert Lyon <robertl@catalyst.net.nz>

Robert Lyon authored Jul 21, 2016 and Aaron Wells committed Jul 25, 2016

It is meant to return artefact instances but was missing the plugin name

behatnotneeded: Testing with phpunit instead

Change-Id: Id12df696b932d3644e85fe6836a3d97910bf36ff
Signed-off-by: Robert Lyon <robertl@catalyst.net.nz>

Robert Lyon authored Dec 18, 2015 and Aaron Wells committed Jul 25, 2016

This got added in 2010 via
commit 87a2c5f2



but we don't need it now, and will end up complicating the ability to
do views in multiple collections, so will remove

behatnotneeded  - existing tests should cover

Change-Id: Ie39a72442cae10727bf9f26ed86050e66e7df4bd
Signed-off-by: Robert Lyon <robertl@catalyst.net.nz>
Signed-off-by: Aaron Wells <aaronw@catalyst.net.nz>

Part of the change is to remove duplicate view access rows as there is
no need to have duplicates of the exact same access for a view.

Adding the id to view_access will allow us to have different view
access for a view in multiple collections in the future

behatnotneeded - should be covered by existing tests

Change-Id: I8d3d6f0a011d0ed01a5ff2931e8e16d3000f4f16
Signed-off-by: Robert Lyon <robertl@catalyst.net.nz>
Signed-off-by: Aaron Wells <aaronw@catalyst.net.nz>

To be more inline with bootstrap stylings

Also passing in $view object so we can find the user's default theme
if the view's one doesn't exist rather than defaulting to 'default'

behatnotneeded

Change-Id: Ia4cc0552604698a1fd1aa2f6de666e82b6b86564
Signed-off-by: Robert Lyon <robertl@catalyst.net.nz>

Aaron Wells authored Jul 06, 2016 and Robert Lyon committed Jul 07, 2016

Since we now have multiple places that check whether the view is a site template
page, I've consolidated the logic for it into View::is_site_template()

Change-Id: Ibf23fe14d7f5bba8e1dc5a1281a2534d32b9887a
behatnotneeded: Test to be provided later

Rather than copy the same artefact once per page we should only copy
it once per copy of page(s) transaction. Eg if we are to copy a
collection of 5 pages and they all have a block pointing to the same
image we should copy that image only once not 5 times.

behatnotneeded - behat file to come

Change-Id: Iecdde53515cdd9d5ee02918252b486aa0f662fab
Signed-off-by: Robert Lyon <robertl@catalyst.net.nz>

behatnotneeded

Change-Id: I64480575d5bb5bfd53a65c26e766e01ae38df510
Signed-off-by: Robert Lyon <robertl@catalyst.net.nz>

Aaron Wells authored Dec 17, 2015 and Robert Lyon committed Jun 21, 2016

Improving speed of sql query by using 'EXISTS' to check against group
views

behatnotneeded

Change-Id: I0546092c5715a582ad6079f875aeac54a9dd0255
Signed-off-by: Robert Lyon <robertl@catalyst.net.nz>

Should be $this rather than $view as we are within a view object

Change-Id: I3132d6d380a82c3cd4cbc6fc35a4ab5fb19af673
Signed-off-by: Robert Lyon <robertl@catalyst.net.nz>

Bug 1580499. Updating comments feature to adapt to the new
behavior, and removing redundant "feedback_configuration" feature
file.

Change-Id: Ib48cbb19f6ab9cc4937f31cef504724569680e1f

Whether they are a group / institution blog or not

bahatnotneeded

Change-Id: I34b73e57db58108ef149019a5419c9ee5ca69e9f
Signed-off-by: Robert Lyon <robertl@catalyst.net.nz>

behatnotneeded

Change-Id: I9b0f9e4e804cdd582a70ace810ae09fc4ff84606
Signed-off-by: Robert Lyon <robertl@catalyst.net.nz>

Kristina Hoeppner authored May 21, 2016 and Robert Lyon committed May 26, 2016

in various places around Mahara to be more consistent
with what people see on the interface.

Change-Id: I83e088d39964ecd1d72945973fd0b4995b82da65

Made a centeralised function so that we can use 'viewsperpage' only
for the different pages lists. And have 'itemsperpage' for certain
admin lists.

behatnotneeded

Change-Id: Ie8b49bdc0d86ef49f66e9a1bf09799e6bc21484b
Signed-off-by: Robert Lyon <robertl@catalyst.net.nz>

Liam Sharpe authored Apr 22, 2016 and Robert Lyon committed May 06, 2016

Note: the buttons are currently not displayed as a button group

behatnotneeded

Change-Id: I5d2e3cfe0bcdd8017885b058ec5cf33c61583846