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

Spoiler images of owned caches are missing the description #5221

Closed
Lineflyer opened this issue Oct 5, 2015 · 5 comments
Closed

Spoiler images of owned caches are missing the description #5221

Lineflyer opened this issue Oct 5, 2015 · 5 comments
Assignees
Labels
Bug Issues classified as a bug
Milestone

Comments

@Lineflyer
Copy link
Member

From support mail (but not yet verified):

With Version 2015.09.28 you fixed "Spoiler image descripritions". This is working now for each Cache of other Cachers. But descriptions of OWNED caches are still missing. I think the reason is that the HTML-code of owned caches is slightly different

the following code is additional for owned caches:

<span class="minorDetails"> | <a href="/seek/image.aspx?imgid=......" 
title="Edit">Edit</a></span>

Maybe you can make image descriptions of owned Caches working too, thanks :-)

@Lineflyer Lineflyer added Bug Issues classified as a bug Unverified Issue not yet confirmed/reproduced or feature requests not yet checked for plausibility labels Oct 5, 2015
@Lineflyer
Copy link
Member Author

Confirmed for my owned cache GC352Y3.
I added a description to the spoiler to test it and can't see it.
If I access the cache as another user with c:geo the description is shown.

@Lineflyer Lineflyer removed the Unverified Issue not yet confirmed/reproduced or feature requests not yet checked for plausibility label Mar 29, 2016
@Bananeweizen
Copy link
Member

Can you send the HTML page of your own cache to my account or the list? I don't want to fiddle with the regular expressions without having a test case.

@Lineflyer
Copy link
Member Author

@Bananeweizen Sure, will do next minutes.

@Lineflyer Lineflyer assigned Bananeweizen and unassigned Lineflyer Apr 3, 2016
@Bananeweizen Bananeweizen added the Field test Issue to be (re)tested in the field label Apr 24, 2016
@Bananeweizen
Copy link
Member

please test that with a nightly, since I had quite some trouble to get my unit tests running

@Bananeweizen Bananeweizen reopened this Apr 24, 2016
@Lineflyer
Copy link
Member Author

Tested and OK with 2016.04.24-NB1

Tested using my owned cache, same as initially.

@Lineflyer Lineflyer removed the Field test Issue to be (re)tested in the field label Apr 24, 2016
@Lineflyer Lineflyer assigned Bananeweizen and unassigned Lineflyer Apr 24, 2016
@Lineflyer Lineflyer added this to the Master branch milestone Apr 24, 2016
@Lineflyer Lineflyer modified the milestones: 2016.08.14, Master branch Aug 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Issues classified as a bug
Projects
None yet
Development

No branches or pull requests

2 participants