Skip to content

Commit cd5cbd0

Browse files
authored
Merge pull request #59 from Boehringer-Ingelheim/test
v 4.3.2 release
2 parents edbb32c + 0897e98 commit cd5cbd0

59 files changed

Lines changed: 9870 additions & 1166 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.Rbuildignore

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,3 +8,6 @@
88
^\.github
99
^doc$
1010
^Meta$
11+
^\.git-blame-ignore-revs$
12+
^\.lintr.R$
13+
\.gitempty$

DESCRIPTION

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
Package: dv.listings
22
Type: Package
33
Title: Data listings module
4-
Version: 4.3.1
4+
Version: 4.3.2
55
Authors@R:
66
c(
77
person("Boehringer-Ingelheim Pharma GmbH & Co.KG", role = c("cph", "fnd")),
@@ -26,7 +26,8 @@ Suggests:
2626
pharmaverseadam (>= 0.1.0),
2727
shinytest2 (>= 0.3.1),
2828
testthat (>= 3.0.2),
29-
safetyData
29+
safetyData,
30+
jsonlite
3031
Config/testthat/edition: 3
3132
Imports:
3233
checkmate (>= 2.1.0),
@@ -41,13 +42,11 @@ Imports:
4142
shinyFeedback (>= 0.4.0),
4243
shinyjs (>= 2.1.0),
4344
shinyWidgets (>= 0.8.0),
44-
tibble (>= 3.2.1),
45-
xxhashlite,
4645
base64enc,
47-
htmltools,
48-
jsonlite
46+
htmltools
4947
Depends: R (>= 4.1.0)
5048
RoxygenNote: 7.3.2
5149
Roxygen: list(markdown = TRUE)
52-
Remotes: boehringer-ingelheim/dv.manager@main
50+
Remotes: boehringer-ingelheim/dv.manager
5351
VignetteBuilder: knitr
52+
URL: https://github.com/Boehringer-Ingelheim/dv.listings, https://boehringer-ingelheim.github.io/dv.listings

LICENSE

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
Copyright 2024 Boehringer-Ingelheim Pharma GmbH & Co.KG
1+
Copyright 2024-2026 Boehringer-Ingelheim Pharma GmbH & Co.KG
22

33
Licensed under the Apache License, Version 2.0 (the "License");
44
you may not use this file except in compliance with the License.

NAMESPACE

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,3 +8,4 @@ export(mod_listings)
88
import(shiny)
99
importFrom(shiny,NS)
1010
importFrom(shiny,tagList)
11+
useDynLib(dv.listings, .registration=TRUE)

NEWS.md

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,13 @@
1+
# dv.listings 4.3.2
2+
3+
- Limited support for column width resizing
4+
- Remove dependencies on jsonlite and lastpage.sty latex package
5+
- Review functionality:
6+
- Per-role undo of review actions
7+
- Faster response to review actions
8+
- Progress indicators during long review actions
9+
- Improved checks for asynchronous javascript-mediated review write actions
10+
111
# dv.listings 4.3.1
212

313
- Guard against selection of nested or conflicting review storage location.

R/CM.R

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -790,7 +790,7 @@ CM <- local({ # _C_hecked _M_odule
790790

791791
df_to_string <- function(df) {
792792
names(df) <- sprintf("[%s] ", names(df))
793-
lines <- capture.output(print(as.data.frame(df), right = FALSE, row.names = FALSE, quote = TRUE)) |> trimws()
793+
lines <- utils::capture.output(print(as.data.frame(df), right = FALSE, row.names = FALSE, quote = TRUE)) |> trimws()
794794
return(paste(lines, collapse = "\n"))
795795
}
796796

R/aaa_preface.R

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
11
# THIS FILE IS NAMED aaa_preface.R SO IT IS LOADED BEFORE ALL OTHER FILES
22
# DO NOT CHANGE ITS NAME. IT MUST BE THE FIRST ONE ALPHABETICALLY.
33

4+
#' @useDynLib dv.listings, .registration=TRUE
5+
NULL
6+
47
#' Build a collection of named constants
58
#'
69
#' @param ... Named parameters to be collected as constants

R/export_helpers.R

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -351,7 +351,7 @@ pdf_preprocessing <- function(df, ref) {
351351

352352
# Create index list which contains for every page the start and end row and the start and end column
353353
index_list <- lapply(1:n_rows_pages, function(i) {
354-
return(lapply(seq_len(length(start_new_page)), function(j) {
354+
return(lapply(seq_along(start_new_page), function(j) {
355355
ind_i <- ifelse(i < n_rows_pages, i * PDF_EXP$N_ROWS, nrow(df))
356356
ind_j <- ifelse(j < length(start_new_page), start_new_page[j + 1] - 1, ncol(df))
357357
return(c((i - 1) * PDF_EXP$N_ROWS + 1, ind_i, start_new_page[j], ind_j))

0 commit comments

Comments
 (0)