You've already forked revanced-patcher
mirror of
https://github.com/revanced/revanced-patcher
synced 2025-09-06 16:38:50 +02:00
Compare commits
6 Commits
v18.0.0-de
...
v19.0.0
Author | SHA1 | Date | |
---|---|---|---|
![]() |
671aa6d507 | ||
![]() |
b697bbad2b | ||
![]() |
f05a404e48 | ||
![]() |
a46e948b5a | ||
![]() |
dc09ea639f | ||
![]() |
49ed096e85 |
73
CHANGELOG.md
73
CHANGELOG.md
@@ -1,3 +1,76 @@
|
||||
# [19.0.0](https://github.com/ReVanced/revanced-patcher/compare/v18.0.0...v19.0.0) (2023-10-24)
|
||||
|
||||
|
||||
### Features
|
||||
|
||||
* Add `PatchOption#valueType` to handle type erasure ([a46e948](https://github.com/ReVanced/revanced-patcher/commit/a46e948b5a0cf9bc8d31f557e371cd7d7c2f5b1c))
|
||||
|
||||
|
||||
### BREAKING CHANGES
|
||||
|
||||
* This changes the signature of the `PatchOption` constructor.
|
||||
|
||||
# [19.0.0-dev.1](https://github.com/ReVanced/revanced-patcher/compare/v18.0.0...v19.0.0-dev.1) (2023-10-24)
|
||||
|
||||
|
||||
### Features
|
||||
|
||||
* Add `PatchOption#valueType` to handle type erasure ([a46e948](https://github.com/ReVanced/revanced-patcher/commit/a46e948b5a0cf9bc8d31f557e371cd7d7c2f5b1c))
|
||||
|
||||
|
||||
### BREAKING CHANGES
|
||||
|
||||
* This changes the signature of the `PatchOption` constructor.
|
||||
|
||||
# [18.0.0](https://github.com/ReVanced/revanced-patcher/compare/v17.0.0...v18.0.0) (2023-10-22)
|
||||
|
||||
|
||||
### Bug Fixes
|
||||
|
||||
* Do not set patch fields if they are empty ([a76ac04](https://github.com/ReVanced/revanced-patcher/commit/a76ac04214a2ab91e3b2f9dddb13ed52816fe723))
|
||||
* Only allow setting `MethodFingerprint#result` privately ([aed1eac](https://github.com/ReVanced/revanced-patcher/commit/aed1eac3157317acf87f522750cf2f41509606c3))
|
||||
|
||||
|
||||
### Code Refactoring
|
||||
|
||||
* Change `PatchOption` from abstract to open class ([09cd6aa](https://github.com/ReVanced/revanced-patcher/commit/09cd6aa568988dd5241bfa6a2e12b7926a7b0683))
|
||||
* Change data classes to actual classes ([6192089](https://github.com/ReVanced/revanced-patcher/commit/6192089b71bdca15765369f3e607ddd1f8266205))
|
||||
* Convert extension functions to member functions ([e2ca507](https://github.com/ReVanced/revanced-patcher/commit/e2ca50729da7085799c0ff6fc4f7afaf82579738))
|
||||
* Move files to simplify package structure ([124a2e9](https://github.com/ReVanced/revanced-patcher/commit/124a2e9d3efb88f0f038ae306d941e918ad3ad3c))
|
||||
* Remove deprecated classes and members ([a4212f6](https://github.com/ReVanced/revanced-patcher/commit/a4212f6bf952971541c4550e20f6bf57a382e19a))
|
||||
|
||||
|
||||
* refactor!: Remove `Fingerprint` interface ([54a2f8f](https://github.com/ReVanced/revanced-patcher/commit/54a2f8f16fddf2b2ed47eb23717ba3734c4a6c5d))
|
||||
|
||||
|
||||
### Features
|
||||
|
||||
* Add function to reset options to their default value ([ebbaafb](https://github.com/ReVanced/revanced-patcher/commit/ebbaafb78e88f34faeafe9ff8532afe29231bd79))
|
||||
* Add function to reset options to their default value ([e6de90d](https://github.com/ReVanced/revanced-patcher/commit/e6de90d300bc9c82ca1696cb898db04c65a1cd5b))
|
||||
* Add getter for default option value ([c7922e9](https://github.com/ReVanced/revanced-patcher/commit/c7922e90d0c6ae83f513611c706ebea33c1a2b63))
|
||||
* Make `PatchOption#values` nullable ([56ce9ec](https://github.com/ReVanced/revanced-patcher/commit/56ce9ec2f98ff351c3d42df71b49e5c88f07e665))
|
||||
* Name patch option value validator property correctly ([caa634f](https://github.com/ReVanced/revanced-patcher/commit/caa634fac6d7a717f54e3b015827c8858fd637b9))
|
||||
* Remove patch annotation processor ([4456031](https://github.com/ReVanced/revanced-patcher/commit/445603145979a6f67823a79f9d6cd140299cff37))
|
||||
* Use a map for `PatchOption#values` ([54ac139](https://github.com/ReVanced/revanced-patcher/commit/54ac1394a914d3eed7865ec697e8016834134911))
|
||||
|
||||
|
||||
### Performance Improvements
|
||||
|
||||
* Run the garbage collector after writing dex files ([d9fb241](https://github.com/ReVanced/revanced-patcher/commit/d9fb241d57b0c4340130c0e5900250e66730ea56))
|
||||
|
||||
|
||||
### BREAKING CHANGES
|
||||
|
||||
* The `MethodFingerprint#result` member can now only be set inside `MethodFingerprint`.
|
||||
* The `Fingerprint` interface is no longer present.
|
||||
* Some extension functions are now member functions.
|
||||
* This gets rid of data class members.
|
||||
* Some deprecated classes and members are not present anymore.
|
||||
* Classes and members have changed packages.
|
||||
* This gets rid of the existing basic implementations of the `PatchOptions` type and moves extension functions.
|
||||
* This changes the getter name of the property.
|
||||
* Various patch constructor signatures have changed.
|
||||
|
||||
# [18.0.0-dev.6](https://github.com/ReVanced/revanced-patcher/compare/v18.0.0-dev.5...v18.0.0-dev.6) (2023-10-22)
|
||||
|
||||
|
||||
|
@@ -289,7 +289,7 @@ public abstract interface annotation class app/revanced/patcher/patch/annotation
|
||||
|
||||
public class app/revanced/patcher/patch/options/PatchOption {
|
||||
public static final field PatchExtensions Lapp/revanced/patcher/patch/options/PatchOption$PatchExtensions;
|
||||
public fun <init> (Ljava/lang/String;Ljava/lang/Object;Ljava/util/Map;Ljava/lang/String;Ljava/lang/String;ZLkotlin/jvm/functions/Function2;)V
|
||||
public fun <init> (Ljava/lang/String;Ljava/lang/Object;Ljava/util/Map;Ljava/lang/String;Ljava/lang/String;ZLjava/lang/String;Lkotlin/jvm/functions/Function2;)V
|
||||
public final fun getDefault ()Ljava/lang/Object;
|
||||
public final fun getDescription ()Ljava/lang/String;
|
||||
public final fun getKey ()Ljava/lang/String;
|
||||
@@ -298,6 +298,7 @@ public class app/revanced/patcher/patch/options/PatchOption {
|
||||
public final fun getValidator ()Lkotlin/jvm/functions/Function2;
|
||||
public final fun getValue ()Ljava/lang/Object;
|
||||
public final fun getValue (Ljava/lang/Object;Lkotlin/reflect/KProperty;)Ljava/lang/Object;
|
||||
public final fun getValueType ()Ljava/lang/String;
|
||||
public final fun getValues ()Ljava/util/Map;
|
||||
public fun reset ()V
|
||||
public final fun setValue (Ljava/lang/Object;)V
|
||||
|
@@ -1,4 +1,4 @@
|
||||
org.gradle.parallel = true
|
||||
org.gradle.caching = true
|
||||
kotlin.code.style = official
|
||||
version = 18.0.0-dev.6
|
||||
version = 19.0.0
|
||||
|
@@ -12,6 +12,7 @@ import kotlin.reflect.KProperty
|
||||
* @param title The title.
|
||||
* @param description A description.
|
||||
* @param required Whether the option is required.
|
||||
* @param valueType The type of the option value (to handle type erasure).
|
||||
* @param validator The function to validate the option value.
|
||||
* @param T The value type of the option.
|
||||
*/
|
||||
@@ -23,6 +24,7 @@ open class PatchOption<T>(
|
||||
val title: String?,
|
||||
val description: String?,
|
||||
val required: Boolean,
|
||||
val valueType: String,
|
||||
val validator: PatchOption<T>.(T?) -> Boolean
|
||||
) {
|
||||
/**
|
||||
@@ -110,7 +112,9 @@ open class PatchOption<T>(
|
||||
description: String? = null,
|
||||
required: Boolean = false,
|
||||
validator: PatchOption<String>.(String?) -> Boolean = { true }
|
||||
) = PatchOption(key, default, values, title, description, required, validator).also { registerOption(it) }
|
||||
) = PatchOption(
|
||||
key, default, values, title, description, required, "String", validator
|
||||
).also { registerOption(it) }
|
||||
|
||||
/**
|
||||
* Create a new [PatchOption] with an integer value and add it to the current [Patch].
|
||||
@@ -135,7 +139,9 @@ open class PatchOption<T>(
|
||||
description: String? = null,
|
||||
required: Boolean = false,
|
||||
validator: PatchOption<Int?>.(Int?) -> Boolean = { true }
|
||||
) = PatchOption(key, default, values, title, description, required, validator).also { registerOption(it) }
|
||||
) = PatchOption(
|
||||
key, default, values, title, description, required, "Int", validator
|
||||
).also { registerOption(it) }
|
||||
|
||||
/**
|
||||
* Create a new [PatchOption] with a boolean value and add it to the current [Patch].
|
||||
@@ -160,7 +166,9 @@ open class PatchOption<T>(
|
||||
description: String? = null,
|
||||
required: Boolean = false,
|
||||
validator: PatchOption<Boolean?>.(Boolean?) -> Boolean = { true }
|
||||
) = PatchOption(key, default, values, title, description, required, validator).also { registerOption(it) }
|
||||
) = PatchOption(key, default, values, title, description, required, "Boolean", validator).also {
|
||||
registerOption(it)
|
||||
}
|
||||
|
||||
/**
|
||||
* Create a new [PatchOption] with a float value and add it to the current [Patch].
|
||||
@@ -185,7 +193,9 @@ open class PatchOption<T>(
|
||||
description: String? = null,
|
||||
required: Boolean = false,
|
||||
validator: PatchOption<Float?>.(Float?) -> Boolean = { true }
|
||||
) = PatchOption(key, default, values, title, description, required, validator).also { registerOption(it) }
|
||||
) = PatchOption(
|
||||
key, default, values, title, description, required, "Float", validator
|
||||
).also { registerOption(it) }
|
||||
|
||||
/**
|
||||
* Create a new [PatchOption] with a long value and add it to the current [Patch].
|
||||
@@ -210,7 +220,9 @@ open class PatchOption<T>(
|
||||
description: String? = null,
|
||||
required: Boolean = false,
|
||||
validator: PatchOption<Long?>.(Long?) -> Boolean = { true }
|
||||
) = PatchOption(key, default, values, title, description, required, validator).also { registerOption(it) }
|
||||
) = PatchOption(
|
||||
key, default, values, title, description, required, "Long", validator
|
||||
).also { registerOption(it) }
|
||||
|
||||
/**
|
||||
* Create a new [PatchOption] with a string array value and add it to the current [Patch].
|
||||
@@ -235,7 +247,9 @@ open class PatchOption<T>(
|
||||
description: String? = null,
|
||||
required: Boolean = false,
|
||||
validator: PatchOption<Array<String>?>.(Array<String>?) -> Boolean = { true }
|
||||
) = PatchOption(key, default, values, title, description, required, validator).also { registerOption(it) }
|
||||
) = PatchOption(
|
||||
key, default, values, title, description, required, "StringArray", validator
|
||||
).also { registerOption(it) }
|
||||
|
||||
/**
|
||||
* Create a new [PatchOption] with an integer array value and add it to the current [Patch].
|
||||
@@ -260,7 +274,9 @@ open class PatchOption<T>(
|
||||
description: String? = null,
|
||||
required: Boolean = false,
|
||||
validator: PatchOption<Array<Int>?>.(Array<Int>?) -> Boolean = { true }
|
||||
) = PatchOption(key, default, values, title, description, required, validator).also { registerOption(it) }
|
||||
) = PatchOption(
|
||||
key, default, values, title, description, required, "IntArray", validator
|
||||
).also { registerOption(it) }
|
||||
|
||||
/**
|
||||
* Create a new [PatchOption] with a boolean array value and add it to the current [Patch].
|
||||
@@ -285,7 +301,9 @@ open class PatchOption<T>(
|
||||
description: String? = null,
|
||||
required: Boolean = false,
|
||||
validator: PatchOption<Array<Boolean>?>.(Array<Boolean>?) -> Boolean = { true }
|
||||
) = PatchOption(key, default, values, title, description, required, validator).also { registerOption(it) }
|
||||
) = PatchOption(
|
||||
key, default, values, title, description, required, "BooleanArray", validator
|
||||
).also { registerOption(it) }
|
||||
|
||||
/**
|
||||
* Create a new [PatchOption] with a float array value and add it to the current [Patch].
|
||||
@@ -310,7 +328,9 @@ open class PatchOption<T>(
|
||||
description: String? = null,
|
||||
required: Boolean = false,
|
||||
validator: PatchOption<Array<Float>?>.(Array<Float>?) -> Boolean = { true }
|
||||
) = PatchOption(key, default, values, title, description, required, validator).also { registerOption(it) }
|
||||
) = PatchOption(
|
||||
key, default, values, title, description, required, "FloatArray", validator
|
||||
).also { registerOption(it) }
|
||||
|
||||
/**
|
||||
* Create a new [PatchOption] with a long array value and add it to the current [Patch].
|
||||
@@ -335,7 +355,9 @@ open class PatchOption<T>(
|
||||
description: String? = null,
|
||||
required: Boolean = false,
|
||||
validator: PatchOption<Array<Long>?>.(Array<Long>?) -> Boolean = { true }
|
||||
) = PatchOption(key, default, values, title, description, required, validator).also { registerOption(it) }
|
||||
) = PatchOption(
|
||||
key, default, values, title, description, required, "LongArray", validator
|
||||
).also { registerOption(it) }
|
||||
|
||||
private fun <P : Patch<*>> P.registerOption(option: PatchOption<*>) = option.also { options.register(it) }
|
||||
}
|
||||
|
@@ -85,6 +85,10 @@ internal class PatchOptionsTest {
|
||||
}
|
||||
}
|
||||
|
||||
@Test
|
||||
fun `option types should be known`() =
|
||||
assertTrue(OptionsTestPatch.options["array"].valueType == "StringArray")
|
||||
|
||||
@Test
|
||||
fun `getting default value should work`() =
|
||||
assertDoesNotThrow { assertNull(OptionsTestPatch.resettableOption.default) }
|
||||
|
Reference in New Issue
Block a user