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

Newly created filter through the element picker is ignored after quitting/launching uBO #2585

Closed
gorhill opened this issue May 6, 2017 · 1 comment

Comments

@gorhill
Copy link
Owner

gorhill commented May 6, 2017

Describe the issue

When a certain number of conditions are met, creating a filter through the element picker can cause that filter to not be applied next time uBO is launched.

Conditions:

  • There must be a valid selfie in use.

    • A selfie is created after 23 minutes where no filter changes occur (updating lists, editing My filters, changing the set of lists, etc).
  • Once a selfie is present and valid, a filter must be created through the element picker.

  • The user must quit uBO before 23 minutes has elapsed.

One or more specific URLs where the issue occurs

http://blog.planet5d.com/

Steps for anyone to reproduce the issue

  • Wait for a selfie to be present and valid (leave uBO idle for 24 minutes).
  • Open http://blog.planet5d.com/ (no existing filter at time of writing to take care of these ads)
  • Open the element picker and add ##.g to the input field.
    • As a result, all the ads on the page will be highlighted
  • Click the Create button.
    • Ads disappear. Forcing a reload confirms that the new filter works fine.
  • Quit uBO.
  • Launch uBO.
  • Navigate to the web page above.

Result: ads are showing. The filter last created is seemingly ignored. This is caused by the fact that uBO used the non-invalidated selfie to load its data, and the new filter is not part of that selfie, which should have been invalidated when the filter was created.

Your settings

Default settings.

  • OS/version: Any.
  • Browser/version: Any.
  • uBlock Origin version: 1.12.3.
Your filter lists

Default lists.

@gorhill gorhill changed the title Selfie not immediately invalidated following creation of filter(s) through element picker Newly created filter through the element picker is ignored after quitting/launching uBO May 6, 2017
@gorhill
Copy link
Owner Author

gorhill commented May 6, 2017

This is a regression bug as a result of asset management refactoring. The invalidation of the selfie used to be done properly in the current scenario:

this.selfieManager.destroy();
.

@gorhill gorhill closed this as completed in fe61cdd May 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant