Poor style progress update commit

This commit is contained in:
Timothy Warren 2015-11-13 16:31:01 -05:00
parent 83cd815750
commit afced2339a
12 changed files with 124 additions and 8 deletions

1
.gitignore vendored
View File

@ -3,6 +3,7 @@
*.workspace *.workspace
vendor vendor
app/cache/* app/cache/*
app/logs/*
public/images/* public/images/*
public/js/cache/* public/js/cache/*
composer.lock composer.lock

View File

@ -9,6 +9,10 @@ use Aura\Html\HelperLocatorFactory;
use Aura\Web\WebFactory; use Aura\Web\WebFactory;
use Aura\Router\RouterFactory; use Aura\Router\RouterFactory;
use Aura\Session\SessionFactory; use Aura\Session\SessionFactory;
use Monolog\Logger;
use Monolog\Handler\RotatingFileHandler;
use Monolog\Handler\BrowserConsoleHandler;
use Aviat\Ion\Di\Container; use Aviat\Ion\Di\Container;
use Aviat\AnimeClient\Auth\HummingbirdAuth; use Aviat\AnimeClient\Auth\HummingbirdAuth;
@ -18,6 +22,15 @@ use Aviat\AnimeClient\Auth\HummingbirdAuth;
return function(array $config_array = []) { return function(array $config_array = []) {
$container = new Container(); $container = new Container();
// -------------------------------------------------------------------------
// Logging
// -------------------------------------------------------------------------
$app_logger = new Logger('animeclient');
$app_logger->pushHandler(new RotatingFileHandler(__DIR__.'/logs/app.log', Logger::NOTICE));
$app_logger->pushHandler(new BrowserConsoleHandler(Logger::DEBUG));
$container->setLogger($app_logger);
// ------------------------------------------------------------------------- // -------------------------------------------------------------------------
// Injected Objects // Injected Objects
// ------------------------------------------------------------------------- // -------------------------------------------------------------------------

View File

@ -9,6 +9,7 @@ return [
'on_hold' => '/on_hold', 'on_hold' => '/on_hold',
'dropped' => '/dropped', 'dropped' => '/dropped',
'completed' => '/completed', 'completed' => '/completed',
//'collection' => '/collection/view',
'all' => '/all' 'all' => '/all'
] ]
], ],

0
app/logs/.gitkeep Normal file
View File

View File

@ -8,12 +8,12 @@
<section class="media-wrap"> <section class="media-wrap">
<?php foreach($items as $item): ?> <?php foreach($items as $item): ?>
<article class="media" id="manga-<?= $item['id'] ?>"> <article class="media" id="manga-<?= $item['id'] ?>">
<?php /*if (is_logged_in()): ?> <?php if ($auth->is_authenticated()): ?>
<div class="edit_buttons" hidden> <div class="edit_buttons" hidden>
<button class="plus_one_chapter">+1 Chapter</button> <button class="plus_one_chapter">+1 Chapter</button>
<button class="plus_one_volume">+1 Volume</button> <button class="plus_one_volume">+1 Volume</button>
</div> </div>
<?php endif*/ ?> <?php endif ?>
<img src="<?= $escape->attr($item['manga']['image']) ?>" /> <img src="<?= $escape->attr($item['manga']['image']) ?>" />
<div class="name"> <div class="name">
<a href="<?= $item['manga']['url'] ?>"> <a href="<?= $item['manga']['url'] ?>">
@ -45,6 +45,6 @@
<?php endforeach ?> <?php endforeach ?>
<?php endif ?> <?php endif ?>
</main> </main>
<?php /*if (is_logged_in()): ?> <?php if ($auth->is_authenticated()): ?>
<script src="<?= $urlGenerator->asset_url('js.php?g=edit') ?>"></script> <script src="<?= $urlGenerator->asset_url('js.php?g=edit') ?>"></script>
<?php endif*/ ?> <?php endif ?>

View File

@ -13,6 +13,8 @@
"danielstjules/stringy": "~2.1", "danielstjules/stringy": "~2.1",
"filp/whoops": "dev-php7#fe32a402b086b21360e82013e8a0355575c7c6f4", "filp/whoops": "dev-php7#fe32a402b086b21360e82013e8a0355575c7c6f4",
"guzzlehttp/guzzle": "6.*", "guzzlehttp/guzzle": "6.*",
"monolog/monolog": "1.*",
"psr/log": "~1.0",
"robmorgan/phinx": "0.4.*", "robmorgan/phinx": "0.4.*",
"szymach/c-pchart": "1.*" "szymach/c-pchart": "1.*"
} }

View File

@ -80,6 +80,6 @@ $container->set('error-handler', $defaultHandler);
// ----------------------------------------------------------------------------- // -----------------------------------------------------------------------------
// Dispatch to the current route // Dispatch to the current route
// ----------------------------------------------------------------------------- // -----------------------------------------------------------------------------
$container->get('dispatcher')(); $container->get('dispatcher')->__invoke();
// End of index.php // End of index.php

0
public/js/collection.js Normal file → Executable file
View File

14
public/js/manga_edit.js Normal file → Executable file
View File

@ -1,7 +1,9 @@
/** /**
* Javascript for editing manga, if logged in * Javascript for editing manga, if logged in
*/ */
(function ($, undefined) { (function ($) {
"use strict";
if (CONTROLLER !== "manga") return; if (CONTROLLER !== "manga") return;
@ -28,10 +30,18 @@
// Update the total count // Update the total count
data[type + "s_read"] = ++completed; data[type + "s_read"] = ++completed;
$.post(BASE_URL + 'update', data, function(res) { $.ajax({
data: data,
dataType: 'json',
method: 'POST',
mimeType: 'application/json',
url: BASE_URL + '/' + CONTROLLER + '/update'
}).done(function(res) {
console.table(res); console.table(res);
parent_sel.find("."+type+"s_read").text(completed); parent_sel.find("."+type+"s_read").text(completed);
add_message('success', "Sucessfully updated " + res.manga[0].romaji_title); add_message('success', "Sucessfully updated " + res.manga[0].romaji_title);
}).fail(function() {
add_message('error', "Failed to updated " + res.manga[0].romaji_title);
}); });
}); });

View File

@ -3,6 +3,7 @@
namespace Aviat\Ion\Di; namespace Aviat\Ion\Di;
use ArrayObject; use ArrayObject;
use Psr\Log\LoggerInterface;
/** /**
* Dependency container * Dependency container
@ -16,6 +17,13 @@ class Container implements ContainerInterface {
*/ */
protected $container = []; protected $container = [];
/**
* Map of logger instances
*
* @var ArrayObject
*/
protected $loggers = [];
/** /**
* Constructor * Constructor
* *
@ -24,6 +32,7 @@ class Container implements ContainerInterface {
public function __construct(array $values = []) public function __construct(array $values = [])
{ {
$this->container = new ArrayObject($values); $this->container = new ArrayObject($values);
$this->loggers = new ArrayObject([]);
} }
/** /**
@ -56,7 +65,7 @@ class Container implements ContainerInterface {
* *
* @param string $id * @param string $id
* @param mixed $value * @param mixed $value
* @return Container * @return ContainerInterface
*/ */
public function set($id, $value) public function set($id, $value)
{ {
@ -76,5 +85,41 @@ class Container implements ContainerInterface {
{ {
return $this->container->offsetExists($id); return $this->container->offsetExists($id);
} }
/**
* Determine whether a logger channel is registered
* @param string $key The logger channel
* @return boolean
*/
public function hasLogger($key = 'default')
{
return $this->loggers->offsetExists($key);
}
/**
* Add a logger to the Container
*
* @param LoggerInterface $logger
* @param string $key The logger 'channel'
* @return ContainerInterface
*/
public function setLogger(LoggerInterface $logger, $key = 'default')
{
$this->loggers[$key] = $logger;
return $this;
}
/**
* Retrieve a logger for the selected channel
*
* @param string $key The logger to retreive
* @return LoggerInterface|null
*/
public function getLogger($key = 'default')
{
return ($this->hasLogger($key))
? $this->loggers[$key]
: NULL;
}
} }
// End of Container.php // End of Container.php

View File

@ -2,6 +2,8 @@
namespace Aviat\Ion\Di; namespace Aviat\Ion\Di;
use Psr\Log\LoggerInterface;
/** /**
* Interface for the Dependency Injection Container * Interface for the Dependency Injection Container
*/ */
@ -15,4 +17,20 @@ interface ContainerInterface extends \Interop\Container\ContainerInterface {
* @return ContainerInterface * @return ContainerInterface
*/ */
public function set($key, $value); public function set($key, $value);
/**
* Add a logger to the Container
*
* @param LoggerInterface $logger
* @param string $key The logger 'channel'
*/
public function setLogger(LoggerInterface $logger, $key = 'default');
/**
* Retrieve a logger for the selected channel
*
* @param string $key The logger to retreive
* @return LoggerInterface|null
*/
public function getLogger($key = 'default');
} }

View File

@ -2,6 +2,9 @@
use Aviat\Ion\Di\Container; use Aviat\Ion\Di\Container;
use Aviat\Ion\Di\Exception\ContainerException; use Aviat\Ion\Di\Exception\ContainerException;
use Monolog\Logger;
use Monolog\Handler\TestHandler;
use Monolog\Handler\NullHandler;
class ContainerTest extends AnimeClient_TestCase { class ContainerTest extends AnimeClient_TestCase {
@ -48,4 +51,27 @@ class ContainerTest extends AnimeClient_TestCase {
} }
} }
public function testLoggerMethods()
{
// Does the container have the default logger?
$this->assertFalse($this->container->hasLogger());
$this->assertFalse($this->container->hasLogger('default'));
$logger1 = new Logger('default');
$logger2 = new Logger('testing');
$logger1->pushHandler(new NullHandler());
$logger2->pushHandler(new TestHandler());
// Set the logger channels
$this->container->setLogger($logger1);
$this->container->setLogger($logger2, 'test');
$this->assertEquals($logger1, $this->container->getLogger('default'));
$this->assertEquals($logger2, $this->container->getLogger('test'));
$this->assertNull($this->container->getLogger('foo'));
$this->assertTrue($this->container->hasLogger());
$this->assertTrue($this->container->hasLogger('default'));
$this->assertTrue($this->container->hasLogger('test'));
}
} }