Comments (3)
Actually, I'm definitely adding those hooks. In loadRoutes
, I'll probably just add parable_routing_load_before
and parable_routing_load_after
like you suggested. Also a good moment to add hooks to loadInits
🙂
from parable.
I'm assuming you mean you don't want to add them all into \Routing\App
? 😉
Good suggestions!
And I get why, I've had a similar issue, but I haven't gotten around to implementing a proper fix for it yet, since most projects I do in Parable tend to max out at around 20-30 routes or so.
What I've done in those cases is the following, which I can suggest as a work-around:
namespace Routing;
class App extends \Parable\Framework\Routing\Base
{
/** @var array */
protected $combinedRoutes = [];
public function __construct(
\Routing\Front $routingFront,
\Routing\Admin $routingAdmin,
\Routing\Dashboard $routingDashboard
) {
$this->combinedRoutes = array_merge(
$routingFront->get(),
$routingAdmin->get(),
$routingDashboard->get()
);
}
public function get()
{
return $this->combinedRoutes;
}
}
\Routing\App
is DI-ed, so this should work. Mind you, this is a pseudo-code version, I don't have the actual code handy at the moment, but I think you'll see that this could be an option until there's an 'official' way to do it 🙂
from parable.
I like the work-around, thanks! 😄
from parable.
Related Issues (20)
- SQL errors not thrown by PDOMySQL class HOT 3
- To install in doc root or parent? HOT 2
- Typo in docs: initLocations should be inits? HOT 2
- `\Parable\Console\Parameter` can't handle `--option=value` HOT 4
- Make `\Parable\Http\Response` HTTP/2 ready HOT 2
- redirectToRoute does not accept parameters HOT 1
- Json->prepare() should have check on valid json_encode HOT 2
- MySQL database connection might require charset HOT 2
- Request could use a getBody() HOT 1
- Http\Url::getUrl('/') gives domain plus index.php HOT 13
- GetSet that uses php://input? HOT 7
- This project needs a cool icon... HOT 3
- Code style consistency before V1 release HOT 2
- Console/Output: cursor and linelength HOT 4
- Call to undefined function posix_isatty()
- It's 2018, fix LICENSE and README HOT 1
- Add all 'parable.*' to `\Config\App` structure file HOT 2
- Dispatcher & Response don't handle mixed-type content very well HOT 2
- ConsoleApp instantiates and prepares every available command HOT 3
- extractParameterValues checks for empty(), disabling value '0' HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from parable.