search

nemein / com_meego_packages

MeeGo Community Applications browser
3 stars 1 forks source link

Strange notice on top of the page #21

Closed feri closed 13 years ago

feri commented 13 years ago

Go to: http://apps-beta.meego.com/basecategories/meego/current/handset/

A notice appears on the top: Notice: Undefined offset: 1 in /var/cas_latest/midgardmvc_core/route.php on line 220

I don't remember seeing it before upgrading to Midgard 10.05.5-1 (dev branch).

feri commented 13 years ago

Upgrading midgardmvc_core solved the issue. Could we have more regression testing in MVC Core?

bergie commented 13 years ago

I'm happy to accept additional tests to https://github.com/midgardproject/midgardmvc_core/tree/master/tests

Current status is Tests: 103, Assertions: 364, Failures: 1, Skipped: 1. with the failure and skip coming from the new observation service.

bergie commented 13 years ago

Seems this is still happening. I can see the notice on every other request.

feri commented 13 years ago

On 04/14/2011 04:14 PM, bergie wrote:

Seems this is still happening. I can see the notice on every other request.

Yes, because we have templates that are simply wrong, but never cleaned up from the cache... If you do shift reload a few times then the notice disappears.. and then comes back again...

bergie commented 13 years ago

Try with the patch I mentioned in https://github.com/midgardproject/midgardmvc_core/issues/52#issuecomment-1001231

bergie commented 13 years ago

Also, run /mgd:cache/invalidate

feri commented 13 years ago

On 04/14/2011 04:57 PM, bergie wrote:

Also, run /mgd:cache/invalidate

I am pretty sure other apps have the same issue. Just take a look at any cache/midgarmvc/template directory and see the mess. I am doing cleanups by hand, still.. Anyway I think this should be rather discussed at the other issue I opened for mvc core.

feri commented 13 years ago

This is gone, or at least I don't see it.