Re: [Openinteract-dev] Bug in base_page module

[Chris W. <ch...@cw...>]

Teemu wrote:
> page actions -> directory index has a bug. "map new directory handler" which
> seems to be create action for PageDirectory doesn't work. If you enter a new
> directory over there and press Modify, you get "Error: Object modification
> failed. Error found:". I looked at it a little bit and noticed that a new
> page is not created if old_directory parameter is empty (currently a hidden
> field on the Create New Directory Handler-page). If I forge the URL to be
> something like:
> 
> /PageDirectory/create/?old_directory=/whatever
> 
> .. and then type the directory name I want, it will work and create a new
> directory.
> 
> If you edit some directory, you can also change the Directory location, 
> resulting in a new directory handler. This obviously doesn't sound like 
> editing but creating a new one. Maybe in edit mode editing the directory 
> location should be restricted.

You know what's even weirder? I re-created this bug and got the same 
error as you, but when I tried to immediately re-run the same action it 
worked! What's even stranger is that the first time I see the following 
in the log:

OpenInteract::Handler::GenericDispatcher::handler (32) >> Trying to run
    [MySite::Handler::PageDirectory] with [edit]
OpenInteract::CommonHandler::_edit_assign_fields (593) >> Object edit:
    (page_directory) (directory) ()
OpenInteract::CommonHandler::_edit_assign_fields (593) >> Object edit:
    (page_directory) (action) ()
OpenInteract::CommonHandler::_edit_assign_fields (601) >> Object edit
    toggle: (page_directory) (subdirs_inherit) (no)

Those second and third sets of lines are what concern me -- it looks 
like the data aren't being passed back properly or something.... Here's 
what it should look like (what I got on the next attempt):

OpenInteract::Handler::GenericDispatcher::handler (32) >> Trying to run
    [MySite::Handler::PageDirectory] with [edit]
OpenInteract::CommonHandler::_edit_assign_fields (593) >> Object edit:
    (page_directory) (directory) (/foo)
OpenInteract::CommonHandler::_edit_assign_fields (593) >> Object edit:
    (page_directory) (action) (simple_index)
OpenInteract::CommonHandler::_edit_assign_fields (601) >> Object edit
    toggle: (page_directory) (subdirs_inherit) (yes)

More investigation is clearly required here...

> Also, another bug I found occurs when you create a page in location
> /test/test.html and then try to create a page in location /test.
> 
> /test will be created but the content you enter doesn't display at all and it
> acts strangely and it's impossible to remove /test after creating it unless 
> you do it manually (if you do search page and go to /test you can modify it, 
> but if you press remove it displays an empty page instead of removing it).

More on this soon...

Chris

-- 
Chris Winters (ch...@cw...)
Building enterprise-capable snack solutions since 1988.