Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

XMLFilterBase has bogus startElementNS #40158

Closed
mlh mannequin opened this issue Apr 16, 2004 · 3 comments
Closed

XMLFilterBase has bogus startElementNS #40158

mlh mannequin opened this issue Apr 16, 2004 · 3 comments
Assignees

Comments

@mlh
Copy link
Mannequin

mlh mannequin commented Apr 16, 2004

BPO 936637
Nosy @loewis

Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

Show more details

GitHub fields:

assignee = 'https://github.com/loewis'
closed_at = <Date 2004-05-06.02:05:12.000>
created_at = <Date 2004-04-16.22:05:25.000>
labels = ['expert-XML']
title = 'XMLFilterBase has bogus startElementNS'
updated_at = <Date 2004-05-06.02:05:12.000>
user = 'https://bugs.python.org/mlh'

bugs.python.org fields:

activity = <Date 2004-05-06.02:05:12.000>
actor = 'loewis'
assignee = 'loewis'
closed = True
closed_date = None
closer = None
components = ['XML']
creation = <Date 2004-04-16.22:05:25.000>
creator = 'mlh'
dependencies = []
files = []
hgrepos = []
issue_num = 936637
keywords = []
message_count = 3.0
messages = ['20517', '20518', '20519']
nosy_count = 2.0
nosy_names = ['loewis', 'mlh']
pr_nums = []
priority = 'high'
resolution = 'fixed'
stage = None
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue936637'
versions = ['Python 2.3']

@mlh
Copy link
Mannequin Author

mlh mannequin commented Apr 16, 2004

The version in XMLFilterBase is:

    def startElementNS(self, name, qname, attrs):
        self._cont_handler.startElement(name, attrs)

It should be:

    def startElementNS(self, name, qname, attrs):
        self._cont_handler.startElementNS(name, qname,
attrs)

I've seen examples using this method, so in some
version somewhere it must work -- but not in my 2.4
standard libraries, at least.

@mlh mlh mannequin closed this as completed Apr 16, 2004
@mlh mlh mannequin assigned loewis Apr 16, 2004
@mlh mlh mannequin added the topic-XML label Apr 16, 2004
@mlh mlh mannequin closed this as completed Apr 16, 2004
@mlh mlh mannequin assigned loewis Apr 16, 2004
@mlh mlh mannequin added the topic-XML label Apr 16, 2004
@mlh
Copy link
Mannequin Author

mlh mannequin commented Apr 16, 2004

Logged In: YES
user_id=20535

Err.. Make that: "not in my 2.3 standard libraries"

@loewis
Copy link
Mannequin

loewis mannequin commented May 6, 2004

Logged In: YES
user_id=21627

Thanks for pointing that out. Fixed in

saxutils.py 1.21.10.1, 1.22
NEWS 1.831.4.104

Fix in PyXML is pending.

@ezio-melotti ezio-melotti transferred this issue from another repository Apr 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

0 participants