Skip to content

Commit 07ef1d5

Browse files
Call functions from models.DatafileHandler
Call resource updating functions directly from models.DatafileHandler instead of calling super(). Signed-off-by: Ayan Sinha Mahapatra <[email protected]>
1 parent ef5ca31 commit 07ef1d5

File tree

12 files changed

+24
-32
lines changed

12 files changed

+24
-32
lines changed

src/packagedcode/alpine.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ def compute_normalized_license(cls, package):
129129

130130
@classmethod
131131
def assign_package_to_resources(cls, package, resource, codebase):
132-
super().assign_package_to_parent_tree(
132+
models.DatafileHandler.assign_package_to_parent_tree(
133133
package=package,
134134
resource=resource,
135135
codebase=codebase,

src/packagedcode/build.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ def parse(cls, location):
5151

5252
@classmethod
5353
def assign_package_to_resources(cls, package, resource, codebase):
54-
super().assign_package_to_parent_tree(
54+
models.DatafileHandler.assign_package_to_parent_tree(
5555
package=package,
5656
resource=resource,
5757
codebase=codebase,
@@ -234,7 +234,7 @@ class BazelBuildHandler(BaseStarlarkManifestHandler):
234234

235235
@classmethod
236236
def assign_package_to_resources(cls, package, resource, codebase, skip_name='BUILD'):
237-
return super().assign_package_to_resources(
237+
return models.DatafileHandler.assign_package_to_resources(
238238
package=package,
239239
resource=resource,
240240
codebase=codebase,
@@ -251,7 +251,7 @@ class BuckPackageHandler(BaseStarlarkManifestHandler):
251251

252252
@classmethod
253253
def assign_package_to_resources(cls, package, resource, codebase, skip_name='BUCK'):
254-
return super().assign_package_to_resources(
254+
return models.DatafileHandler.assign_package_to_resources(
255255
package=package,
256256
resource=resource,
257257
codebase=codebase,
@@ -376,7 +376,7 @@ def compute_normalized_license(cls, package):
376376

377377
@classmethod
378378
def assign_package_to_resources(cls, package, resource, codebase):
379-
super().assign_package_to_parent_tree(
379+
models.DatafileHandler.assign_package_to_parent_tree(
380380
package_=package,
381381
resource=resource,
382382
codebase=codebase,

src/packagedcode/build_gradle.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ def parse(cls, location):
3535
# TODO: handle complex cases of nested builds with many packages
3636
@classmethod
3737
def assign_package_to_resources(cls, package, resource, codebase):
38-
super().assign_package_to_parent_tree(
38+
models.DatafileHandler.assign_package_to_parent_tree(
3939
package=package,
4040
resource=resource,
4141
codebase=codebase,

src/packagedcode/conda.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ def get_conda_root(cls, resource, codebase):
7171

7272
@classmethod
7373
def assign_package_to_resources(cls, package, resource, codebase):
74-
return super().assign_package_to_resources(
74+
return models.DatafileHandler.assign_package_to_resources(
7575
package=package,
7676
resource=cls.get_conda_root(resource, codebase),
7777
codebase=codebase,

src/packagedcode/debian.py

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ def parse(cls, location):
6767
@classmethod
6868
def assign_package_to_resources(cls, package, resource, codebase):
6969
# only assign this resource
70-
return super().assign_package_to_resources(package, resource, codebase)
70+
return models.DatafileHandler.assign_package_to_resources(package, resource, codebase)
7171

7272

7373
# TODO: introspect archive
@@ -92,7 +92,7 @@ def parse(cls, location):
9292
@classmethod
9393
def assign_package_to_resources(cls, package, resource, codebase):
9494
# only assign this resource
95-
return super().assign_package_to_resources(package, resource, codebase)
95+
return models.DatafileHandler.assign_package_to_resources(package, resource, codebase)
9696

9797

9898
# TODO: introspect archive
@@ -117,7 +117,7 @@ def parse(cls, location):
117117
@classmethod
118118
def assign_package_to_resources(cls, package, resource, codebase):
119119
# only assign this resource
120-
return super().assign_package_to_resources(package, resource, codebase)
120+
return models.DatafileHandler.assign_package_to_resources(package, resource, codebase)
121121

122122

123123
# TODO: also look into neighboring md5sum and data.tarball copyright files!!!
@@ -142,7 +142,7 @@ def assign_package_to_resources(cls, package, resource, codebase):
142142
# two levels up
143143
root = resource.parent(codebase).parent(codebase)
144144
if root:
145-
return super().assign_package_to_resources(package, root, codebase)
145+
return models.DatafileHandler.assign_package_to_resources(package, root, codebase)
146146

147147

148148
# TODO: also look into neighboring copyright files!!!
@@ -169,7 +169,7 @@ def assign_package_to_resources(cls, package, resource, codebase):
169169
# two levels up
170170
root = resource.parent(codebase).parent(codebase)
171171
if root:
172-
return super().assign_package_to_resources(package, root, codebase)
172+
return models.DatafileHandler.assign_package_to_resources(package, root, codebase)
173173

174174

175175
class DebianDscFileHandler(models.DatafileHandler):
@@ -198,7 +198,7 @@ def parse(cls, location):
198198
@classmethod
199199
def assign_package_to_resources(cls, package, resource, codebase):
200200
# only assign this resource
201-
return super().assign_package_to_resources(package, resource, codebase)
201+
return models.DatafileHandler.assign_package_to_resources(package, resource, codebase)
202202

203203

204204
class DebianInstalledStatusDatabaseHandler(models.DatafileHandler):
@@ -489,7 +489,7 @@ def assign_package_to_resources(cls, package, resource, codebase):
489489
# two levels up
490490
root = resource.parent(codebase).parent(codebase)
491491
if root:
492-
return super().assign_package_to_resources(package, root, codebase)
492+
return models.DatafileHandler.assign_package_to_resources(package, root, codebase)
493493

494494

495495
def build_package_data_from_package_filename(filename, datasource_id, package_type,):

src/packagedcode/godeps.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ def parse(cls, location):
7575

7676
@classmethod
7777
def assign_package_to_resources(cls, package, resource, codebase):
78-
super().assign_package_to_parent_tree(package, resource, codebase)
78+
models.DatafileHandler.assign_package_to_parent_tree(package, resource, codebase)
7979

8080

8181
@attr.s

src/packagedcode/jar_manifest.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ def assign_package_to_resources(cls, package, resource, codebase):
5353
parent = resource.parent(codebase)
5454

5555
if parent:
56-
super().assign_package_to_resources(package, resource=parent, codebase=codebase)
56+
models.DatafileHandler.assign_package_to_resources(package, resource=parent, codebase=codebase)
5757

5858

5959

src/packagedcode/maven.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ def assign_package_to_resources(cls, package, resource, codebase):
115115

116116
if resource.path.endswith('.pom'):
117117
# we only treat the parent as the root
118-
return super().assign_package_to_parent_tree(package, resource, codebase)
118+
return models.DatafileHandler.assign_package_to_parent_tree(package, resource, codebase)
119119

120120
# the root is either the parent or further up for poms stored under
121121
# a META-INF dir
@@ -139,7 +139,7 @@ def assign_package_to_resources(cls, package, resource, codebase):
139139
if not root:
140140
root = resource.parent(codebase)
141141

142-
return super().assign_package_to_resources(package, resource=root, codebase=codebase)
142+
return models.DatafileHandler.assign_package_to_resources(package, resource=root, codebase=codebase)
143143

144144
@classmethod
145145
def compute_normalized_license(cls, package):

src/packagedcode/opam.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,7 @@ def parse(cls, location):
119119

120120
@classmethod
121121
def assign_package_to_resources(cls, package, resource, codebase):
122-
return super().assign_package_to_parent_tree(package, resource, codebase)
122+
return models.DatafileHandler.assign_package_to_parent_tree(package, resource, codebase)
123123

124124

125125
def get_repository_homepage_url(name):

src/packagedcode/phpcomposer.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ def assemble(cls, package_data, resource, codebase):
4545

4646
@classmethod
4747
def assign_package_to_resources(cls, package, resource, codebase):
48-
return super().assign_package_to_parent_tree(package, resource, codebase)
48+
return models.DatafileHandler.assign_package_to_parent_tree(package, resource, codebase)
4949

5050
@classmethod
5151
def compute_normalized_license(cls, package):

0 commit comments

Comments
 (0)