Launchpad itself

Merge ~cjwatson/launchpad:drop-native-string-adapter into launchpad:master

  1. Git
  2. lp:~cjwatson/launchpad
  3. drop-native-string-adapter
  4. Merge into master
Proposed by Colin Watson
Status: Merged
Approved by: Colin Watson
Approved revision: 7272d867dcea60db6e1ac752c8d1aafdaacdff8b
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~cjwatson/launchpad:drop-native-string-adapter
Merge into: launchpad:master
Diff against target: 20 lines (+0/-9)
1 file modified
lib/lp/app/browser/configure.zcml (+0/-9)
Reviewer Review Type Date Requested Status
Ioana Lasc (community) Approve
Review via email: mp+407103@code.launchpad.net

Commit message

Drop INativeString formatting adapter registration

To post a comment you must log in.
Revision history for this message
Ioana Lasc (ilasc) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/lib/lp/app/browser/configure.zcml b/lib/lp/app/browser/configure.zcml
2index 005968c..753ce04 100644
3--- a/lib/lp/app/browser/configure.zcml
4+++ b/lib/lp/app/browser/configure.zcml
5@@ -962,15 +962,6 @@
6 />
7
8 <!-- TALES fmt: namespace for strings -->
9- <!-- On Python 2, registering this adapter for both INativeString and
10- ITextString covers both str (bytes) and unicode. On Python 3, this
11- covers only str; the redundancy is harmless. -->
12- <adapter
13- for="zope.interface.common.builtins.INativeString"
14- provides="zope.traversing.interfaces.IPathAdapter"
15- factory="lp.app.browser.stringformatter.FormattersAPI"
16- name="fmt"
17- />
18 <adapter
19 for="zope.interface.common.builtins.ITextString"
20 provides="zope.traversing.interfaces.IPathAdapter"

Subscribers

People subscribed via source and target branches

to status/vote changes: