Skip to content

Commit d48030e

Browse files
authored
Merge pull request #2538 from strictdoc-project/stanislaw/validation
chore: rename: DocumentCachingIterator -> SDocDocumentIterator
2 parents 4b6f14f + 48e8fd0 commit d48030e

File tree

19 files changed

+50
-52
lines changed

19 files changed

+50
-52
lines changed

docs/sdoc_project_statistics.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
from strictdoc.backend.sdoc_source_code.models.source_file_info import (
1010
SourceFileTraceabilityInfo,
1111
)
12-
from strictdoc.core.document_iterator import DocumentCachingIterator
12+
from strictdoc.core.document_iterator import SDocDocumentIterator
1313
from strictdoc.core.project_config import ProjectConfig
1414
from strictdoc.core.statistics.metric import Metric, MetricSection
1515
from strictdoc.core.traceability_index import TraceabilityIndex
@@ -85,7 +85,7 @@ def export(
8585
document_tree_stats.git_commit_hash = git_client.get_commit_hash()
8686

8787
for document in traceability_index.document_tree.document_list:
88-
document_iterator = DocumentCachingIterator(document)
88+
document_iterator = SDocDocumentIterator(document)
8989
for node, _ in document_iterator.all_content(print_fragments=False):
9090
if isinstance(node, SDocNode) and node.node_type == "SECTION":
9191
document_tree_stats.total_sections += 1

strictdoc/backend/reqif/p01_sdoc/sdoc_to_reqif_converter.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@
5151
ParentReqReference,
5252
Reference,
5353
)
54-
from strictdoc.core.document_iterator import DocumentCachingIterator
54+
from strictdoc.core.document_iterator import SDocDocumentIterator
5555
from strictdoc.core.document_tree import DocumentTree
5656
from strictdoc.helpers.cast import assert_cast
5757
from strictdoc.helpers.ordered_set import OrderedSet
@@ -259,7 +259,7 @@ def convert_document_tree(
259259
)
260260
spec_types.extend(document_spec_types)
261261

262-
document_iterator = DocumentCachingIterator(document)
262+
document_iterator = SDocDocumentIterator(document)
263263

264264
# FIXME: This is a throw-away object. It gets discarded when the
265265
# iteration is over. Find a way to do without it.

strictdoc/backend/sdoc/models/document.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -320,10 +320,10 @@ def build_search_index(self) -> None:
320320
map_nodes_by_mid = {}
321321

322322
from strictdoc.core.document_iterator import ( # noqa: PLC0415
323-
DocumentCachingIterator,
323+
SDocDocumentIterator,
324324
)
325325

326-
document_iterator = DocumentCachingIterator(self)
326+
document_iterator = SDocDocumentIterator(self)
327327

328328
for node, _ in document_iterator.all_content(
329329
print_fragments=False,

strictdoc/backend/sdoc/writer.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@
3232
ParentReqReference,
3333
Reference,
3434
)
35-
from strictdoc.core.document_iterator import DocumentCachingIterator
35+
from strictdoc.core.document_iterator import SDocDocumentIterator
3636
from strictdoc.core.document_meta import DocumentMeta
3737
from strictdoc.core.project_config import ProjectConfig
3838
from strictdoc.helpers.cast import assert_cast
@@ -74,7 +74,7 @@ def write_with_fragments(
7474
) -> Tuple[str, Dict[str, str]]:
7575
fragments_dict: Dict[str, str] = {}
7676

77-
document_iterator = DocumentCachingIterator(document)
77+
document_iterator = SDocDocumentIterator(document)
7878
output = ""
7979

8080
output += "[DOCUMENT]"
@@ -295,14 +295,14 @@ def _print_node(
295295
self,
296296
root_node: SDocElementIF,
297297
document: SDocDocument,
298-
document_iterator: DocumentCachingIterator,
298+
document_iterator: SDocDocumentIterator,
299299
) -> str:
300300
# Currently, auto-generated nodes are never written back to file system.
301301
# We could revisit this in the future.
302302
if root_node.autogen:
303303
return ""
304304

305-
assert isinstance(document_iterator, DocumentCachingIterator), (
305+
assert isinstance(document_iterator, SDocDocumentIterator), (
306306
document_iterator
307307
)
308308

strictdoc/core/analyzers/document_uid_analyzer.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
DocumentTreeStats,
1010
SinglePrefixRequirements,
1111
)
12-
from strictdoc.core.document_iterator import DocumentCachingIterator
12+
from strictdoc.core.document_iterator import SDocDocumentIterator
1313
from strictdoc.core.document_tree import DocumentTree
1414
from strictdoc.core.traceability_index import TraceabilityIndex
1515
from strictdoc.helpers.cast import assert_cast
@@ -65,7 +65,7 @@ def analyze_document(
6565
document: SDocDocument,
6666
) -> DocumentStats:
6767
this_document_stats = DocumentStats(document)
68-
document_iterator = DocumentCachingIterator(document)
68+
document_iterator = SDocDocumentIterator(document)
6969
for node, _ in document_iterator.all_content():
7070
if isinstance(node, SDocNode) and node.node_type == "SECTION":
7171
if node.reserved_uid is not None:

strictdoc/core/document_iterator.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ def get_level_string(self) -> str:
4545
return ".".join(map(str, self.level_stack))
4646

4747

48-
class DocumentCachingIterator:
48+
class SDocDocumentIterator:
4949
def __init__(self, document: SDocDocument) -> None:
5050
assert isinstance(document, SDocDocument), document
5151

strictdoc/core/file_traceability_index.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@
3838
SourceFileTraceabilityInfo,
3939
)
4040
from strictdoc.core.constants import GraphLinkType
41-
from strictdoc.core.document_iterator import DocumentCachingIterator
41+
from strictdoc.core.document_iterator import SDocDocumentIterator
4242
from strictdoc.core.project_config import ProjectConfig
4343
from strictdoc.core.source_tree import SourceFile
4444
from strictdoc.helpers.cast import assert_cast
@@ -737,7 +737,7 @@ def create_folder_section(
737737

738738
# Iterate over all generated documents to calculate all node levels.
739739
for document_ in documents_with_generated_content:
740-
document_iterator = DocumentCachingIterator(document_)
740+
document_iterator = SDocDocumentIterator(document_)
741741
for _, _ in document_iterator.all_content(
742742
print_fragments=False,
743743
):

strictdoc/core/traceability_index.py

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
)
1919
from strictdoc.core.asset_manager import AssetManager
2020
from strictdoc.core.constants import GraphLinkType
21-
from strictdoc.core.document_iterator import DocumentCachingIterator
21+
from strictdoc.core.document_iterator import SDocDocumentIterator
2222
from strictdoc.core.document_meta import DocumentMeta
2323
from strictdoc.core.document_tree import DocumentTree
2424
from strictdoc.core.file_dependency_manager import FileDependencyManager
@@ -43,14 +43,14 @@ class TraceabilityIndex:
4343
def __init__(
4444
self,
4545
document_tree: DocumentTree,
46-
document_iterators: Dict[SDocDocument, DocumentCachingIterator],
46+
document_iterators: Dict[SDocDocument, SDocDocumentIterator],
4747
file_traceability_index: FileTraceabilityIndex,
4848
graph_database: GraphDatabase,
4949
file_dependency_manager: FileDependencyManager,
5050
):
51-
self._document_iterators: Dict[
52-
SDocDocument, DocumentCachingIterator
53-
] = document_iterators
51+
self._document_iterators: Dict[SDocDocument, SDocDocumentIterator] = (
52+
document_iterators
53+
)
5454
self._file_traceability_index: FileTraceabilityIndex = (
5555
file_traceability_index
5656
)
@@ -77,7 +77,7 @@ def __init__(
7777
)
7878

7979
@property
80-
def document_iterators(self) -> Dict[SDocDocument, DocumentCachingIterator]:
80+
def document_iterators(self) -> Dict[SDocDocument, SDocDocumentIterator]:
8181
return self._document_iterators
8282

8383
def is_small_project(self) -> bool:
@@ -156,7 +156,7 @@ def get_file_traceability_index(self) -> FileTraceabilityIndex:
156156

157157
def get_document_iterator(
158158
self, document: SDocDocument
159-
) -> DocumentCachingIterator:
159+
) -> SDocDocumentIterator:
160160
return self.document_iterators[document]
161161

162162
def get_parent_requirements(self, requirement: SDocNode) -> List[SDocNode]:
@@ -318,7 +318,7 @@ def get_node_by_uid_weak(
318318
) -> Union[SDocDocument, SDocNode, None]:
319319
assert isinstance(uid, str), uid
320320
for document in self.document_tree.document_list:
321-
document_iterator = DocumentCachingIterator(document)
321+
document_iterator = SDocDocumentIterator(document)
322322
for node, _ in document_iterator.all_content(print_fragments=False):
323323
if isinstance(node, SDocDocument):
324324
if node.config.uid == uid:
@@ -1031,7 +1031,7 @@ def clone_to_bundle_document(
10311031
output_document_dir_rel_path=SDocRelativePath(""),
10321032
)
10331033
traceability_index_copy.document_iterators[bundle_document] = (
1034-
DocumentCachingIterator(bundle_document)
1034+
SDocDocumentIterator(bundle_document)
10351035
)
10361036
for document_ in traceability_index_copy.document_tree.document_list:
10371037
# Ignore all included documents. They are anyway included by

strictdoc/core/traceability_index_builder.py

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@
3232
)
3333
from strictdoc.core.constants import GraphLinkType
3434
from strictdoc.core.document_finder import DocumentFinder
35-
from strictdoc.core.document_iterator import DocumentCachingIterator
35+
from strictdoc.core.document_iterator import SDocDocumentIterator
3636
from strictdoc.core.document_tree import DocumentTree
3737
from strictdoc.core.file_dependency_manager import FileDependencyManager
3838
from strictdoc.core.file_traceability_index import FileTraceabilityIndex
@@ -241,9 +241,7 @@ def create_from_document_tree(
241241

242242
# FIXME: Too many things going on below. Would be great to simplify this
243243
# workflow.
244-
d_01_document_iterators: Dict[
245-
SDocDocument, DocumentCachingIterator
246-
] = {}
244+
d_01_document_iterators: Dict[SDocDocument, SDocDocumentIterator] = {}
247245
d_07_file_traceability_index = FileTraceabilityIndex()
248246

249247
graph_database = GraphDatabase(
@@ -435,7 +433,7 @@ def create_from_document_tree(
435433
rhs_node=document_tags,
436434
)
437435

438-
document_iterator = DocumentCachingIterator(document)
436+
document_iterator = SDocDocumentIterator(document)
439437
d_01_document_iterators[document] = document_iterator
440438

441439
for node, _ in document_iterator.all_content(

strictdoc/core/transforms/delete_requirement.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
SDocNodeIF,
77
)
88
from strictdoc.backend.sdoc.models.node import SDocNode
9-
from strictdoc.core.document_iterator import DocumentCachingIterator
9+
from strictdoc.core.document_iterator import SDocDocumentIterator
1010
from strictdoc.core.traceability_index import TraceabilityIndex
1111
from strictdoc.core.transforms.validation_error import (
1212
MultipleValidationErrorAsList,
@@ -29,7 +29,7 @@ def validate(self) -> None:
2929
document: SDocDocument = assert_cast(
3030
self.requirement.get_document(), SDocDocument
3131
)
32-
document_iterator = DocumentCachingIterator(document=document)
32+
document_iterator = SDocDocumentIterator(document=document)
3333
for document_node_, _ in document_iterator.all_node_content(
3434
self.requirement,
3535
print_fragments=True,

0 commit comments

Comments
 (0)