| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
| |
Currently the AdminPage is still responsible for updating changes to any
of the concerts, but I'd like to get that into their respective classes
too. That way the AdminPage will just be a simple class to handle the
layout of the page, while all the specific functionality is in their
own classes.
This is also the first step to be able to reuse the concerts table on
the public end of the site.
|
|
|
|
|
| |
AdminPage now references the database only through the Concert (and
Venue) models.
|
| |
|
|
|
|
|
|
| |
As a user can only be assigned to one role at the time, we remove the
current user from any role that they may have when clearing the
assignment.
|
|
|
|
| |
It did not return any users, but a form so name it for what it does.
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
The original name did not make much sense. The function didn't return a
user, but a dropdown list of users, where the user currently holding the
given role for the given concert was preselected in the list.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
Replace the table with hardcoded strings in the AdminPage class. This
makes it a pure presentation issue, while the statuses themselves are
just mnemonics.
There's one smell here, and that is that the status values and their
textual representation is split across two modules. (Values in Concert,
and textual representation in AdminPage.) This should probably be
addressed later by refactoring both into a separate AccredStatus class
or something.
|
|
|
|
| |
Clicking the buttons don't work quite yet.
|
|
|
|
|
|
|
|
|
|
|
|
| |
Instead of directly accessing the database with a custom query, we now
just use the Concert::find_concerts method to fetch the concerts that
are to be displayed.
This became much easier now that we don't rely on the extra concertlogs
table.
There's still stuff missing to be functionally equivalent to the old
code, but this should be a lot easier to get in place now.
|
| |
|
| |
|
|
|
|
| |
This must have been forgotten in the previous commit.
|
| |
|
| |
|
|
|
|
|
| |
Now use a Concertlog object to render the correct subform instead of
messing with the db directly.
|
| |
|
|
|
|
|
| |
Now queries the user from the concertlogs table instead of going by
generating a form that is thrown away.
|
|
|
|
| |
No functional change, just trying to make sense of it.
|
|
|
|
|
| |
Also add a `get_status` method to the Concertlogs class, returning the
press status for a given concert_id.
|
| |
|
| |
|
|
|
|
|
|
| |
GoglogAdmin_AdminPage::get_allvenues did not get all venues, but
presented a selection control for selecting a venue. Renamed it as
get_venue_selector instead.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This makes the concert a full object containing all relevant info, while
we can still segment the data in the db.
Instead of this:
$concert = GiglogAdmin_Concert::get($concert_id);
$venue = GiglogAdmin_Venue::get($concert->venue());
echo "{$concert->name()} @ {$venue->name()} : {$concert->cdate()}"
You can now do:
$concert = GiglogAdmin_Concert::get($concert_id);
echo "{$concert->name()} @ {$concert->venue()->name()} : {$concert->cdate()}"
And yeah, renamed Concert::find_cid() to Concert::get() and changed it's
semantics somewhat. It now either returns the given concert if it
exists, or NULL if it does not. Simpler function; simpler to use.
|
|\ |
|
| |
| |
| |
| | |
in dropdown
|
| | |
|
| | |
|
|/ |
|
|
|
|
| |
Modified returnuser function to not allow a user to assign multiple slots to themselves
|
|
|
|
| |
CSS for edit form in giglog
|
|\ |
|
| |
| |
| |
| | |
modified assign/unassing/returnuser functions to use dynamic column name instead of 4 different codes for each assignment type
|
|/
|
|
|
|
| |
We probably need some better error handling here. There's a myriad of
reasons why this call could fail, and we might need to communicate the
failure reason somewhere.
|
|
|
|
| |
Modified so that band is no longer in use
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
Refactored band
|
|
|
|
| |
Run `reuse lint` to verify that all material is licensed.
|
|
|
|
|
|
| |
Filter $_POST['edit'] into a boolean we can use to check wether we're
editing or not. Then use this rather than other implicit assumptions to
detect wether we're an edit form or a new concert form.
|
|
|
|
|
| |
Also allow it to default initialize when passing no args. Not entirely
sure about making this interface public.
|
| |
|
| |
|
|
|
|
| |
Adjustments in admin page and import gigs to use the above adjustments
|
| |
|