Merge ~cjwatson/launchpad:drop-native-string-adapter into launchpad: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
diff --git a/lib/lp/app/browser/configure.zcml b/lib/lp/app/browser/configure.zcml
index 005968c..753ce04 100644
--- a/lib/lp/app/browser/configure.zcml
+++ b/lib/lp/app/browser/configure.zcml
@@ -962,15 +962,6 @@
962 />962 />
963963
964 <!-- TALES fmt: namespace for strings -->964 <!-- TALES fmt: namespace for strings -->
965 <!-- On Python 2, registering this adapter for both INativeString and
966 ITextString covers both str (bytes) and unicode. On Python 3, this
967 covers only str; the redundancy is harmless. -->
968 <adapter
969 for="zope.interface.common.builtins.INativeString"
970 provides="zope.traversing.interfaces.IPathAdapter"
971 factory="lp.app.browser.stringformatter.FormattersAPI"
972 name="fmt"
973 />
974 <adapter965 <adapter
975 for="zope.interface.common.builtins.ITextString"966 for="zope.interface.common.builtins.ITextString"
976 provides="zope.traversing.interfaces.IPathAdapter"967 provides="zope.traversing.interfaces.IPathAdapter"

Subscribers

People subscribed via source and target branches

to status/vote changes: