Get rid of zope.app.zapi

Bug #219302 reported by Lorenzo Gil Sanchez
4
Affects Status Importance Assigned to Milestone
Zope 3
Fix Released
Undecided
Unassigned

Bug Description

AFAIK current coding style practices in Zope3 are against the use of zope.app.zapi package. Instead it is recommended to use the packages that zope.app.zapi wrapps, directly.

The purpose of this bug report is to provide a place where I can attach patches (since I'm not a commiter) for every zope.app.* package that use zope.app.zapi so we can get rid of it.

If you think adding so many patches to one bug report is a little bit overkill, please let me know another way of doing so and I'll be happy to do it.

Revision history for this message
Lorenzo Gil Sanchez (lgs) wrote :
Revision history for this message
Philipp von Weitershausen (philikon) wrote : Re: [Bug 219302] [NEW] Get rid of zope.app.zapi

On 18 Apr 2008, at 19:32 , Lorenzo Gil Sanchez wrote:
> Public bug reported:
>
> AFAIK current coding style practices in Zope3 are against the use of
> zope.app.zapi package. Instead it is recommended to use the packages
> that zope.app.zapi wrapps, directly.
>
> The purpose of this bug report is to provide a place where I can
> attach
> patches (since I'm not a commiter) for every zope.app.* package that
> use
> zope.app.zapi so we can get rid of it.
>
> If you think adding so many patches to one bug report is a little bit
> overkill, please let me know another way of doing so and I'll be happy
> to do it.

Can't you just become a commiter? :)

Revision history for this message
Lorenzo Gil Sanchez (lgs) wrote :

Well, I think I may lack some confidence for touching svn.zope.org. That's why I thought about this simple-easy-project to get used to zope3 development standards and procedures.

On the other hand I already have 18 patches and many more will come. It may be a non trivial task for a commiter to apply them all but for the person who is writting them (me) it can be pretty simple to just commit them.

Revision history for this message
Philipp von Weitershausen (philikon) wrote : Re: [Bug 219302] Re: Get rid of zope.app.zapi

On 18 Apr 2008, at 20:06 , Lorenzo Gil Sanchez wrote:
> Well, I think I may lack some confidence for touching svn.zope.org.
> That's why I thought about this simple-easy-project to get used to
> zope3
> development standards and procedures.
>
> On the other hand I already have 18 patches and many more will come.
> It
> may be a non trivial task for a commiter to apply them all but for the
> person who is writting them (me) it can be pretty simple to just
> commit
> them.

Exactly :). Please become a committer.

Revision history for this message
Lorenzo Gil Sanchez (lgs) wrote :

Ok, I won't attach more patches to this bug. Instead I will apply for a svn account and apply the patches myself.

I'll leave the bug open until we get rid of zope.app.zapi

Revision history for this message
Philipp von Weitershausen (philikon) wrote :

On 18 Apr 2008, at 20:54 , Lorenzo Gil Sanchez wrote:
> Ok, I won't attach more patches to this bug. Instead I will apply
> for a
> svn account and apply the patches myself.
>
> I'll leave the bug open until we get rid of zope.app.zapi

While we can get rid of the imports of zope.app.zapi, we can't get rid
of the zope.app.zapi package for backwards compatibility reasons.
Let's just leave it out there to bit rot and make sure that no other
zope.* or zope.app.* package depends on it.

Revision history for this message
Lorenzo Gil Sanchez (lgs) wrote :

I've removed all existence of zope.app.zapi form the zope.* and zope.app.* namespaces so I'll close the bug now.

Changed in zope3:
status: New → Fix Committed
Revision history for this message
Dan Korostelev (nadako) wrote :

The packages without zope.app.zapi were released.

Changed in zope3:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.