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

Package server undefined indexes on malformed xml #1957

Open
emanuele45 opened this issue Feb 3, 2015 · 1 comment
Open

Package server undefined indexes on malformed xml #1957

emanuele45 opened this issue Feb 3, 2015 · 1 comment
Labels
Milestone

Comments

@emanuele45
Copy link
Contributor

 Type of error: Undefined   8: Undefined index: name
    http://www.elkarte.net/community/index.php?action=admin;area=packageservers;sa=browse;server=3
     File: /.../themes/default/PackageServers.template.php
 Line: 210

and

 Type of error: Undefined   8: Undefined index: description
    http://www.elkarte.net/community/index.php?action=admin;area=packageservers;sa=browse;server=3
     File: /.../sources/admin/PackageServers.controller.php
 Line: 320

Source:
http://www.elkarte.net/community/index.php?topic=2370.msg15569#msg15569

Probably the xml was missing some elements, though it would be nice not to send out errors even in that case, so the array (whatever array it is LOL) should probably be initialized with all the expected elements.

@emanuele45 emanuele45 added the bug label Feb 3, 2015
@emanuele45 emanuele45 added this to the 1.1 Beta milestone Feb 3, 2015
@emanuele45 emanuele45 modified the milestones: 1.1 Beta 1, 1.1 beta 2 Sep 13, 2015
@emanuele45 emanuele45 modified the milestones: 1.1 beta 2, 1.1 beta 3 Jul 26, 2016
@emanuele45 emanuele45 modified the milestones: 1.1 beta 4, 1.1 beta 3 Oct 1, 2016
@emanuele45
Copy link
Contributor Author

/me wonders if at some point wouldn't be better to move to json.

I'm not even entirely sure this should be considered a bug, anyway not something I really care to fix for B4 and probably not even 1.1. 👼

@emanuele45 emanuele45 modified the milestones: 2.0, 1.1 beta 4 Dec 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant