summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2009-03-04Remove cruft from APIBharat Mediratta
2009-03-04Redefine the batch API to be very very simple. You callBharat Mediratta
batch::start() before starting a series of events, and batch::stop() when you're done. In batch mode, the notification module will store up pending notifications. When the batch job is complete, it'll send a single digested email to each user for all of her notifications. Updated the scaffold and local_import to use this. Haven't modified SimpleUploader yet.
2009-03-04A variety of cleanups:Bharat Mediratta
* Allow for the "movie" type in all of our text * Try to follow the pattern of mainly only passing ORM objects to the view and let it generate its own text (this becomes even more important when 3rd parties want to customize notification messages) * Rename _send_message to be _notify_subscribers to be more acccurate and have it explicitly take a subject in the API * Use Item_Model::url() in the views instead of hand crafting URLs * Reformat HTML in views * Use $comment->author_xxx() functions instead of replicating that code * Fix several places where we were encoding data by doing ucfirst($item->type) with conditionals where we form the text properly. We should *never* be showing data types to the end user! This is not localizable! Note that this probably breaks the existing batch processing code. I am going to redo that in a subsequent pass.
2009-03-04Allow url() to return absolute urlsBharat Mediratta
2009-03-04Delete test(), it should never have been checked in.Bharat Mediratta
2009-03-04Simplify logic a bit and tweak the visible text.Bharat Mediratta
2009-03-04Indentation and whitespace tweaks.Bharat Mediratta
2009-03-04Fix indentation.Bharat Mediratta
2009-03-04remove unnecessary render()Bharat Mediratta
2009-03-04Move <label> outside of <?= ?> blockBharat Mediratta
2009-03-04Send 1 items added notification per batch of itemsTim Almdal
2009-03-04Forgot to remove a debugging statementTim Almdal
2009-03-04Use Directory IteratorTim Almdal
2009-03-03* Validate that the source path is authorized.Tim Almdal
* Add site warning message if local_import is installed an there is no authorized directories
2009-03-03Remove csrf verification from autocomplete handlerTim Almdal
2009-03-03Only show local_import head stuff (css and js) when the local_importTim Almdal
admin page is shown
2009-03-03Inline the admin view creation that was in helpers/local_import.phpTim Almdal
and remove it. Cleanup unused variables. Rename the method remove() to remove_path()
2009-03-03Use the gallery root directory on the batch::operation call whenTim Almdal
generating random albums and images
2009-03-03Improve the comment about why we skip the first path.Tim Almdal
Change to use access::required
2009-03-03Fix indentation, remove unnnecessary csrf check.Bharat Mediratta
2009-03-03Create a proxy event (gallery_event) which is called when the requestTim Almdal
is completing.
2009-03-03Add the ability for modules to define hooks. The challenge is thatTim Almdal
when the hooks are run, we haven't added all the installed modules to the path, So if a module defines a hook it will never be run. This change runs any module defined hooks as part of the gallery initialization.
2009-03-03Undo r20234Tim Almdal
2009-03-03Remove the === false and === true checksTim Almdal
I really mean it this time
2009-03-03Removed the === false and === true checksTim Almdal
2009-03-03Refactored the batch Api: 1) created a small batch helper class:Tim Almdal
Starting a batch call batch::operation(name, item). In the case of adding photos name = add and item is the parent of the new items. When the operation is finished the batch::end_operation(name) is called. operation and end_operation events are called. Handlers (i.e. item_created) can call batch::in_progress(name) to determine if a batch is being processed.
2009-03-03If backticks (`) are used to delimit the name of the table inTim Almdal
database, Kohana gets confused an appends the prefix outside of the back ticks
2009-03-03Fix issue identified by security review... some table names where notTim Almdal
being translated.
2009-03-02Forgot to update the install.sql when i changed the [] to {} toTim Almdal
identify table names that need substitution.
2009-03-01Simplify the batch api by having the core event handlers forTim Almdal
start_batch and end_batch add and remove the batch id from the session. Modules wishing to do batch processing, just need to fire the start_batch and end_batch events. Other modules that need to be aware of batches (i.e. notifications) just check the session for "batch_id".
2009-02-28The scaffolding, simple_uploader and local_import now call two newTim Almdal
events: start_add_batch and end_add_batch. The parameter is a batch id which is generated on the first add request. The protocol is call the add_photo as many times as required and then call finish when done. Also renamed the add method in local_import to add_photo so it is consistent with simple_uploader
2009-02-28Change the pattern to identify tables that need prefix substitution toTim Almdal
mirror the drupal pattern of using braces {}.
2009-02-28Correct a type and missed a table nameTim Almdal
2009-02-27This implement table prefix for the watermark notification modules (Ticket #68)Tim Almdal
2009-02-27This implements table prefix for all the queries in core, user, exif,Tim Almdal
tag, search, comment and notification modules (Ticket #68) (missed this one)
2009-02-27This implements table prefix for all the queries in core, user, exif,Tim Almdal
tag, search, comment and notification modules (Ticket #68)
2009-02-27Both the command line and web installer installer now supportsTim Almdal
creating tables with a table prefix. There are still some queries that haven't been converted, so don't start using prefixes yet. However, if you do, you can login and modify the user profile.
2009-02-27* Refactor task management methods from admin_maintenance.php toTim Almdal
task.php * Added a owner_id field to the task database * Modified the admin maintenace to show the owner of the task <<**** Requires a reinstallation of core ****>>
2009-02-271) Change the image block so it has the correct url in the anchor tag.Tim Almdal
2) Change the wrapping class to gImageBlock instead of gImage so the quick kinks aren't enabled
2009-02-27Remove unneeded code.Bharat Mediratta
2009-02-27Optimize the code by ditching the count query; we don't need it.Bharat Mediratta
2009-02-27Replace the string [table_name] with {$prefix}table_name. SlowlyTim Almdal
working through setting up the database access to support table prefixes. (Ticket#68) Before going ahead, just wanted to check this approach... whatcha think?
2009-02-27Remove commented codeTim Almdal
Correct unbalanced brackets
2009-02-27Continue the journey of replacing raw sql with ORM or Database methodTim Almdal
calls (Ticket #68)
2009-02-27Continue the journey of replacing raw sql with ORM or Database methodTim Almdal
calls (Ticket #68)
2009-02-26Update to image_block based on bharat's feedbackTim Almdal
1) move the rand_key column into core 2) don't do a max rand, just try to a get a random number less than the current random number if that doesn't successd look the other way
2009-02-26Replace ORM->select(count(*)) with a call to Database::count_recordsTim Almdal
2009-02-26Removed raw update sql and replaced it with Database::update(...)Tim Almdal
calls. (ticket #68)
2009-02-26Remove the commented line $parent->$movie->parent() as the $parentTim Almdal
object was passed in as a parameter.
2009-02-26Use the Database::update function instead of a raw SQL queryTim Almdal