[Agavi-Tickets] [Agavi] #1401: Linebreak in surrounding autoload XML Tags raises an error on including

1 view
Skip to first unread message

Agavi

unread,
Jul 31, 2011, 7:43:57 AM7/31/11
to tic...@lists.agavi.org
#1401: Linebreak in surrounding autoload XML Tags raises an error on including
------------------------------------------+---------------------------------
Reporter: benjamin@… | Owner: david
Type: defect | Status: new
Priority: normal | Milestone: 1.0.7
Component: config | Version: 1.0.6
Severity: normal | Keywords:
Has_patch: 1 |
------------------------------------------+---------------------------------
Sourcecode format did add an linebreak in the config file, more precisely
in surrounding autoload tags, which resulted in an error on including the
class files, because the the path to the file becomes weird
''"/Users/benjamin/Workspaces/Projekte/Agavi/samples/app/
/Users/benjamin/Workspaces/Projekte/Agavi/samples/app/lib/user/AgaviSampleAppUser.class.php
"''.

For example this is not possible:
{{{
<autoload>
%core.app_dir%/lib/foo/Bar.class.php
</autoload>
}}}

The attached patch does a whitespace trim on the filepath value before the
config is exported.

--
Ticket URL: <http://trac.agavi.org/ticket/1401>
Agavi <http://www.agavi.org/>
An MVC Framework for PHP5

_______________________________________________
Agavi Tickets Mailing List
tic...@lists.agavi.org
http://lists.agavi.org/mailman/listinfo/tickets

Agavi

unread,
Jul 31, 2011, 2:31:20 PM7/31/11
to tic...@lists.agavi.org
#1401: Linebreak in surrounding autoload XML Tags raises an error on including
------------------------------------------+---------------------------------
Reporter: benjamin@… | Owner: david
Type: defect | Status: new
Priority: normal | Milestone: 1.0.7
Component: config | Version: 1.0.6
Severity: normal | Keywords:
Has_patch: 1 |
------------------------------------------+---------------------------------

Comment(by david):

The behavior is because the path doesn't start with a slash (but
whitespace instead), so the compiler prepends the app dir.

The problem is that at some point, we'll have to start trimming
everything... ah well...

--
Ticket URL: <http://trac.agavi.org/ticket/1401#comment:1>

Agavi

unread,
Aug 2, 2011, 1:43:52 AM8/2/11
to tic...@lists.agavi.org
#1401: Linebreak in surrounding autoload XML Tags raises an error on including
------------------------------------------+---------------------------------
Reporter: benjamin@… | Owner: david
Type: defect | Status: new
Priority: normal | Milestone: 1.0.7
Component: config | Version: 1.0.6
Severity: normal | Keywords:
Has_patch: 1 |
------------------------------------------+---------------------------------

Comment(by TuseSqueews):

[http://www.blogster.com/buyneopoints211/buy-neopoints Buy Neopoints]

--
Ticket URL: <http://trac.agavi.org/ticket/1401#comment:2>

Agavi

unread,
Dec 22, 2011, 7:19:47 AM12/22/11
to tic...@lists.agavi.org
#1401: Linebreak in surrounding autoload XML Tags raises an error on including
------------------------------------------+---------------------------------
Reporter: benjamin@… | Owner: david
Type: defect | Status: new
Priority: normal | Milestone: 1.0.8

Component: config | Version: 1.0.6
Severity: normal | Keywords:
Has_patch: 1 |
------------------------------------------+---------------------------------
Changes (by david):

* milestone: 1.0.7 => 1.0.8

Reply all
Reply to author
Forward
0 new messages