| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
|
|
| |
- Source files containing a class should only contain _one_ class.
- Source files containing a class should be names class-[name of the
class].php
- Use dashes instead of underscores in file names.
- Fix source file comments
- Some nitpicking...
|
| |
|
|
|
|
| |
Also fix nonce checking.
|
|
|
|
|
|
| |
Also ensure that nonce checking is performed before both adding and
editing concerts, and escape concert data before outputing it in the
form.
|
|
|
|
|
|
| |
Puts everything that belongs together into the same place. This means
the concerts table is getting a tad large, but let's refactor that
later.
|
| |
|
|
|
|
|
| |
- Lacking type info in venue.php
- Potentially dereference null-objects in giglog_admin_page.php
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
Added mailing for when concert status changes. Not sure how to get line break in text mails AND concert status as text
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|\ |
|