434910 – app-admin/webapp-config - various bug fixes

Bug 434910 - app-admin/webapp-config - various bug fixes

Summary: app-admin/webapp-config - various bug fixes

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Current packages (show other bugs)
Hardware:	All Linux

Importance:	Low enhancement (vote)
Assignee:	Anthony Basile

URL:
Whiteboard:
Keywords:	PATCH

Depends on:
Blocks:

Reported:	2012-09-13 15:20 UTC by Vincent Brillault
Modified:	2013-04-22 01:11 UTC (History)
CC List:	3 users (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
Compilation of small fixes (webapp-config-1.20.19-small-modifications.patch,2.88 KB, patch) 2012-09-13 15:20 UTC, Vincent Brillault	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Vincent Brillault 2012-09-13 15:20:41 UTC

Created attachment 323672 [details, diff]
Compilation of small fixes

Here are some modifications that don't have a real impact:
- add a __str__ method on class PermissionMap for debugging purposes
- Fix a wrong debug mesasge in WebappSource
- Replace __fix(i).strip() by __fix(i) in FileType: __fix already strip the input
- Add missing cases in FileType comments

Comment 1 Devan Franchini (RETIRED) gentoo-dev

2013-03-09 03:11:03 UTC

Seems simple enough, I went over the code and everything checks out. I give it the green light.

Basic code cleanup and debugging stuff :)

Comment 2 Anthony Basile gentoo-dev

2013-04-22 01:11:33 UTC

committed