diff --git a/src/allmydata/web/check_results.py b/src/allmydata/web/check_results.py index 82c0a77..39524a9 100644 --- a/src/allmydata/web/check_results.py +++ b/src/allmydata/web/check_results.py @@ -185,7 +185,7 @@ class ResultsBase: target = target + "?output=%s" % output return T.a(href=target)[si_s] -class LiteralCheckResults(rend.Page, ResultsBase): +class LiteralCheckResultsRenderer(rend.Page, ResultsBase): docFactory = getxmlfile("literal-check-results.xhtml") def __init__(self, client): @@ -226,7 +226,7 @@ class CheckerBase: return T.div[T.a(href=return_to)["Return to file/directory."]] return "" -class CheckResults(CheckerBase, rend.Page, ResultsBase): +class CheckResultsRenderer(CheckerBase, rend.Page, ResultsBase): docFactory = getxmlfile("check-results.xhtml") def __init__(self, client, results): @@ -269,7 +269,7 @@ class CheckResults(CheckerBase, rend.Page, ResultsBase): cr = self._render_results(ctx, data) return ctx.tag[cr] -class CheckAndRepairResults(CheckerBase, rend.Page, ResultsBase): +class CheckAndRepairResultsRenderer(CheckerBase, rend.Page, ResultsBase): docFactory = getxmlfile("check-and-repair-results.xhtml") def __init__(self, client, results): @@ -316,7 +316,7 @@ class CheckAndRepairResults(CheckerBase, rend.Page, ResultsBase): return "" -class DeepCheckResults(rend.Page, ResultsBase, ReloadMixin): +class DeepCheckResultsRenderer(rend.Page, ResultsBase, ReloadMixin): docFactory = getxmlfile("deep-check-results.xhtml") def __init__(self, client, monitor): @@ -331,8 +331,8 @@ class DeepCheckResults(rend.Page, ResultsBase, ReloadMixin): si = base32.a2b(name) r = self.monitor.get_status() try: - return CheckResults(self.client, - r.get_results_for_storage_index(si)) + return CheckResultsRenderer(self.client, + r.get_results_for_storage_index(si)) except KeyError: raise WebError("No detailed results for SI %s" % html.escape(name), http.NOT_FOUND) @@ -470,7 +470,7 @@ class DeepCheckResults(rend.Page, ResultsBase, ReloadMixin): runtime = time.time() - req.processing_started_timestamp return ctx.tag["runtime: %s seconds" % runtime] -class DeepCheckAndRepairResults(rend.Page, ResultsBase, ReloadMixin): +class DeepCheckAndRepairResultsRenderer(rend.Page, ResultsBase, ReloadMixin): docFactory = getxmlfile("deep-check-and-repair-results.xhtml") def __init__(self, client, monitor): @@ -483,10 +483,10 @@ class DeepCheckAndRepairResults(rend.Page, ResultsBase, ReloadMixin): # /operation/$OPHANDLE/$STORAGEINDEX provides detailed information # about a specific file or directory that was checked si = base32.a2b(name) - r = self.monitor.get_status() + s = self.monitor.get_status() try: - return CheckAndRepairResults(self.client, - r.get_results_for_storage_index(si)) + results = s.get_results_for_storage_index(si) + return CheckAndRepairResultsRenderer(self.client, results) except KeyError: raise WebError("No detailed results for SI %s" % html.escape(name), http.NOT_FOUND) diff --git a/src/allmydata/web/directory.py b/src/allmydata/web/directory.py index 58fe781..788f339 100644 --- a/src/allmydata/web/directory.py +++ b/src/allmydata/web/directory.py @@ -28,9 +28,9 @@ from allmydata.web.common import text_plain, WebError, \ get_format, get_mutable_type from allmydata.web.filenode import ReplaceMeMixin, \ FileNodeHandler, PlaceHolderNodeHandler -from allmydata.web.check_results import CheckResults, \ - CheckAndRepairResults, DeepCheckResults, DeepCheckAndRepairResults, \ - LiteralCheckResults +from allmydata.web.check_results import CheckResultsRenderer, \ + CheckAndRepairResultsRenderer, DeepCheckResultsRenderer, \ + DeepCheckAndRepairResultsRenderer, LiteralCheckResultsRenderer from allmydata.web.info import MoreInfo from allmydata.web.operations import ReloadMixin from allmydata.web.check_results import json_check_results, \ @@ -421,7 +421,7 @@ class DirectoryNodeHandler(RenderMixin, rend.Page, ReplaceMeMixin): def _maybe_literal(self, res, Results_Class): if res: return Results_Class(self.client, res) - return LiteralCheckResults(self.client) + return LiteralCheckResultsRenderer(self.client) def _POST_check(self, req): # check this directory @@ -430,10 +430,10 @@ class DirectoryNodeHandler(RenderMixin, rend.Page, ReplaceMeMixin): add_lease = boolean_of_arg(get_arg(req, "add-lease", "false")) if repair: d = self.node.check_and_repair(Monitor(), verify, add_lease) - d.addCallback(self._maybe_literal, CheckAndRepairResults) + d.addCallback(self._maybe_literal, CheckAndRepairResultsRenderer) else: d = self.node.check(Monitor(), verify, add_lease) - d.addCallback(self._maybe_literal, CheckResults) + d.addCallback(self._maybe_literal, CheckResultsRenderer) return d def _start_operation(self, monitor, renderer, ctx): @@ -450,10 +450,10 @@ class DirectoryNodeHandler(RenderMixin, rend.Page, ReplaceMeMixin): add_lease = boolean_of_arg(get_arg(ctx, "add-lease", "false")) if repair: monitor = self.node.start_deep_check_and_repair(verify, add_lease) - renderer = DeepCheckAndRepairResults(self.client, monitor) + renderer = DeepCheckAndRepairResultsRenderer(self.client, monitor) else: monitor = self.node.start_deep_check(verify, add_lease) - renderer = DeepCheckResults(self.client, monitor) + renderer = DeepCheckResultsRenderer(self.client, monitor) return self._start_operation(monitor, renderer, ctx) def _POST_stream_deep_check(self, ctx): diff --git a/src/allmydata/web/filenode.py b/src/allmydata/web/filenode.py index 18ffe2d..fd7e7eb 100644 --- a/src/allmydata/web/filenode.py +++ b/src/allmydata/web/filenode.py @@ -19,8 +19,8 @@ from allmydata.web.common import text_plain, WebError, RenderMixin, \ boolean_of_arg, get_arg, should_create_intermediate_directories, \ MyExceptionHandler, parse_replace_arg, parse_offset_arg, \ get_format, get_mutable_type -from allmydata.web.check_results import CheckResults, \ - CheckAndRepairResults, LiteralCheckResults +from allmydata.web.check_results import CheckResultsRenderer, \ + CheckAndRepairResultsRenderer, LiteralCheckResultsRenderer from allmydata.web.info import MoreInfo class ReplaceMeMixin: @@ -285,7 +285,7 @@ class FileNodeHandler(RenderMixin, rend.Page, ReplaceMeMixin): def _maybe_literal(self, res, Results_Class): if res: return Results_Class(self.client, res) - return LiteralCheckResults(self.client) + return LiteralCheckResultsRenderer(self.client) def _POST_check(self, req): verify = boolean_of_arg(get_arg(req, "verify", "false")) @@ -293,10 +293,10 @@ class FileNodeHandler(RenderMixin, rend.Page, ReplaceMeMixin): add_lease = boolean_of_arg(get_arg(req, "add-lease", "false")) if repair: d = self.node.check_and_repair(Monitor(), verify, add_lease) - d.addCallback(self._maybe_literal, CheckAndRepairResults) + d.addCallback(self._maybe_literal, CheckAndRepairResultsRenderer) else: d = self.node.check(Monitor(), verify, add_lease) - d.addCallback(self._maybe_literal, CheckResults) + d.addCallback(self._maybe_literal, CheckResultsRenderer) return d def render_DELETE(self, ctx):